<p>Am 07.03.2016 16:00 schrieb "Maciej Izak" <<a href="mailto:hnb.code@gmail.com">hnb.code@gmail.com</a>>:<br>
><br>
> 2016-03-07 15:50 GMT+01:00 Jonas Maebe <<a href="mailto:jonas.maebe@elis.ugent.be">jonas.maebe@elis.ugent.be</a>>:<br>
>><br>
>> 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.<br>
><br>
><br>
> Thanks :). I will try to commit more regular. Btw. for mentioned initialization, optimization was not that obvious without implementing Initialization/Finalization operators.</p>
<p>Might be, but even then you can commit that separately.</p>
<p>>><br>
>> 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?)<br>
><br>
><br>
> "var aFoo: TFoo" parameter syntax is "Sven approved" ;) </p>
<p>Jonas meant that you didn't use "aFoo" inside the body of the Initialize/Finalize operators. Is that a copy and paste oversight or does that really work? In the latter case: that definitely shouldn't be the case then.</p>
<p>Regards,<br>
Sven</p>