Re: [PHP-DEV] Remove memory leak in PHP5.4 for bug 55971

2012-01-20 Thread Pierrick Charron
Yes :) I sent you an other mail about this bug yesterday to confirm that I can commit it. P. On 20 January 2012 13:05, Stas Malyshev wrote: > Hi! > > >> Nevery mind. Apparently Dmitry committed a patch for this one. > > > I think what he committed was a patch for another problem - double free in

Re: [PHP-DEV] Remove memory leak in PHP5.4 for bug 55971

2012-01-20 Thread Stas Malyshev
Hi! Nevery mind. Apparently Dmitry committed a patch for this one. I think what he committed was a patch for another problem - double free in traits. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/ (408)454-6900 ext. 227 -- PHP Internals - PHP Runtime Developmen

Re: [PHP-DEV] Remove memory leak in PHP5.4 for bug 55971

2012-01-20 Thread Pierrick Charron
Nevery mind. Apparently Dmitry committed a patch for this one. P. On 20 January 2012 08:04, Pierrick Charron wrote: > Ok i'll only commit it on trunk and 5.3 for right now and keep the > patch ready to apply. What about the other one I sent you (60809) ? > > On 20 January 2012 02:10, Stas Malysh

Re: [PHP-DEV] Remove memory leak in PHP5.4 for bug 55971

2012-01-20 Thread Pierrick Charron
Ok i'll only commit it on trunk and 5.3 for right now and keep the patch ready to apply. What about the other one I sent you (60809) ? On 20 January 2012 02:10, Stas Malyshev wrote: > Hi! > > >> I did run the test of bug #55871 and got memory leaks. This patch >> remove them. Could you please rev

Re: [PHP-DEV] Remove memory leak in PHP5.4 for bug 55971

2012-01-19 Thread Stas Malyshev
Hi! I did run the test of bug #55871 and got memory leaks. This patch remove them. Could you please review the patch and if it is Ok i'll commit it to 5.4 Please hold it for now. It's not a critical issue, if we'd have another RC for different reason, we can also have it in as it doesn't look

[PHP-DEV] Remove memory leak in PHP5.4 for bug 55971

2012-01-19 Thread Pierrick Charron
Hi Stas, I did run the test of bug #55871 and got memory leaks. This patch remove them. Could you please review the patch and if it is Ok i'll commit it to 5.4 Thanks Pierrick Index: NEWS === --- NEWS(revision 322482) +++ NEW