[fpc-devel] Feature announcement: Record management operators
Michael Van Canneyt
michael at freepascal.org
Mon Mar 7 16:01:31 CET 2016
On Mon, 7 Mar 2016, Maciej Izak wrote:
> 2016-03-07 15:50 GMT+01:00 Jonas Maebe <jonas.maebe at elis.ugent.be>:
>
>> 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.
>>
>
> Thanks :). I will try to commit more regular. Btw. for mentioned
> initialization, optimization was not that obvious without implementing
> Initialization/Finalization operators.
>
>
>> 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?)
>
>
> "var aFoo: TFoo" parameter syntax is "Sven approved" ;)
In that case, I think you need to force people to use it.
Michael.
More information about the fpc-devel
mailing list