[fpc-devel] Feature announcement: Record management operators

Michael Van Canneyt michael at freepascal.org
Mon Mar 7 15:53:35 CET 2016



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.



More information about the fpc-devel mailing list