Joost van der Sluis wrote:
Op maandag 15-12-2008 om 23:52 uur [tijdzone -0500], schreef Andrew
Haines:
It fixes a really terrible memory leak.

No.

Not for now at least. First wait for the testsuite-results (if there are
no tests for this particular part, then we even have to wait some
longer, as we need users to test it). Then let it settle at least a few
days. Then we'll see.

If it is important then I already tested it after Andrew commit and now using it in the fix compiled lhelp application (where it was visible before and was a reason of this fix). No problems found. And memory dont jump too high as before.

IMO you are using very high restrictions for fpc packages. There must be less restrictive rules for fixes that does not touch compiler, rtl and fcl or developers will move their packages outside fpc as it is now with LNet. Who needs this?

Best regards,
Paul Ishenin.

_______________________________________________
fpc-devel maillist  -  fpc-devel@lists.freepascal.org
http://lists.freepascal.org/mailman/listinfo/fpc-devel

Reply via email to