Grokbase › Groups › PHP › php-internals › January 2012
FAQ
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

Search Discussions

  • Stas Malyshev at Jan 20, 2012 at 7:10 am ⇧
    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 to
    dangerous.
    --
    Stanislav Malyshev, Software Architect
    SugarCRM: http://www.sugarcrm.com/
    (408)454-6900 ext. 227
  • Pierrick Charron at Jan 20, 2012 at 1:04 pm ⇧
    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 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 to
    dangerous.
    --
    Stanislav Malyshev, Software Architect
    SugarCRM: http://www.sugarcrm.com/
    (408)454-6900 ext. 227
  • Pierrick Charron at Jan 20, 2012 at 1:14 pm ⇧
    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 Malyshev wrote:
    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 to
    dangerous.
    --
    Stanislav Malyshev, Software Architect
    SugarCRM: http://www.sugarcrm.com/
    (408)454-6900 ext. 227
  • Stas Malyshev at Jan 20, 2012 at 6:05 pm ⇧
    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
  • Pierrick Charron at Jan 20, 2012 at 6:25 pm ⇧
    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
    traits.

    --
    Stanislav Malyshev, Software Architect
    SugarCRM: http://www.sugarcrm.com/
    (408)454-6900 ext. 227

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupphp-internals @
categoriesphp
postedJan 20, '12 at 3:35a
activeJan 20, '12 at 6:25p
posts6
users2
websitephp.net

2 users in discussion

Pierrick Charron: 4 posts Stas Malyshev: 2 posts

Content

People

Support

Translate

site design / logo © 2022 Grokbase