On Mon, 7 Mar 2016, Jonas Maebe wrote:
Maciej Izak wrote on Mon, 07 Mar 2016:
I'm pleased to finally announce the additional record operators:
Initialize, Finalize.
Congratulations! However, in the future please try to split your commits
more. For example, it seems the optimisation regarding the RTTI for
initialisation could have been committed (and hence tested and potentially
merged) separately from the rest.
The syntax with the separate "var aFoo: TFoo" parameter, but then seemingly
using that parameter as some kind of automatic alias for a fake "self" in the
body, is also rather strange (why don't you have to use "aFoo.F := 10" etc?)
Exactly my question.
I think it would have been preferable to drop the parameter and simply introduce
'self' ?
The current solution seems quite counter-intuitive.
Michael.
_______________________________________________
fpc-devel maillist - fpc-devel@lists.freepascal.org
http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel