<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2018-01-30 14:40 GMT+01:00 Marco van de Voort <span dir="ltr"><<a href="mailto:marcov@stack.nl" target="_blank">marcov@stack.nl</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Have you tested this with large methods? The trouble is we only get<br>
reports/code from people for whom the hack succeeded, not from ones that<br>
tried and failed.<br>
<br>
I can remember from one of the bugreports or mail threads about very large<br>
procedure bodies (where FPC hit virtual register numbers) that Delphi starts<br>
to deallocate temps earlier depending on procedure size and maybe used<br>
language constructs.  (number of structured temps?) That was mostly strings,<br>
but might also go for interfaces. I can also imagine that one would like<br>
to disable this in case recursion is detected.<br></blockquote><div><br></div><div>Yes I am aware of this. In the case of interfaces for large methods all seems works as presented for simple example. I was not able to reproduce this problem for interfaces.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
> Seems like the SCOPEDINTERFACEDESTROY mode-switch is necessary.<br>
<br>
</span>No, one could also simply fix the relevant code.<br></blockquote><div><br></div><div>I am not the fan of SCOPEDINTERFACEDESTROY. My first impression about this was : WTF. In some cases is almost impossible to fix large legacy (and often wrong) code base.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
> question is: would we like to enabling this for all Delphi modes?<br>
<br>
</span>Everything that needs this is essentially feature abuse. I would not enable<br>
it by default</blockquote><div><br></div><div>+1 . Anyway I am still considering : commit or not :P.</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Best regards,<br>Maciej Izak</div></div></div>
</div></div>