<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2016-03-07 15:50 GMT+01:00 Jonas Maebe <span dir="ltr"><<a href="mailto:jonas.maebe@elis.ugent.be" target="_blank">jonas.maebe@elis.ugent.be</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">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></blockquote><div><br></div><div>Thanks :). I will try to commit more regular. Btw. for mentioned initialization, optimization was not that obvious without implementing Initialization/Finalization operators.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
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?)</blockquote><div><br></div><div>"var aFoo: TFoo" parameter syntax is "Sven approved" ;) </div></div>-- <br><div><div dir="ltr"><div>Best regards,<br>Maciej Izak</div></div></div>
</div></div>