This project is archived and is in readonly mode.

#1253 ✓wontfix
Martin K.

Severe performance issues with garbage collection (IE6/IE7)

Reported by Martin K. | July 5th, 2011 @ 10:24 AM

Hi everybody,
since the Mootools Upgrade from Version 1.1 to 1.3 we have problems with a significantly worse performance. When continuimg from a page with many search results, the unload event takes a very long time leading to an additional 4-5 seconds loading time.
We have analyzed the problem and we have come to the conclusion that the following method is responsible for the problems:

/*<ltIE9>*/
if (window.attachEvent && !window.addEventListener) window.addListener('unload', function() {
    Object.each(collected, clean);
    if (window.CollectGarbage) CollectGarbage();
});
/*</ltIE9>*/

Removing this code fragment has resolved the issue, but now we are afraid of unwanted side effects, such as memory leaks.
Is it safe to remove this sniplet? Is there an alternative? What do you recommend?
Thank you for your quick response,

Martin from Berlin

Comments and changes to this ticket

  • fakedarren

    fakedarren July 16th, 2011 @ 04:25 PM

    • State changed from “new” to “wontfix”
    • Assigned user set to “fakedarren”

    Hi Martin

    I wouldn't remove that code if I were you :)

    If you were to I would expect memory consumption to go up quite quickly and performance across pages would drop as a result.

    The length of time it is taking is a concern though. Are you creating a lot of events? If so, you could try to use delegation instead.

    Regards
    Darren

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

Pages