Does JavaScript's setInterval() method cause memory leak?

http://stackoverflow.com/questions/14034107/does-javascript-setinterval-method-cause-memory-leak/

by killahpriest • 13 years ago

23 12 13 years ago

balloot2• 13 years ago

This is silly. You're creating a new anonymous function on every interval, so it will increasingly take up memory to hold these anon functions until the GC kicks in. It's doing exactly what it should do, because you don't want it doing GC every 50 ms.

ajanuary3• 13 years ago

It isn't creating a new anonymous function every interval. It's created once and passed in as an argument.

It's presumably the call stack that the memory is used for.

teraflop• 13 years ago

I don't think that's what's happening. The anonymous function is created just once, before calling setInterval, not within the callback itself.

abstractbill• 13 years ago

That anonymous function is a literal, just like the number 50 (the next argument after it). In the same way the 50 doesn't get created each iteration, neither does the anonymous function.

balloot2• 13 years ago

The memory profile says otherwise, doesn't it? If you create a named function and call it like "setInterval(draw, 50)", the memory use stays constant.

enjo• 13 years ago

My tenuous understanding of the ECMAScript spec would suggest that a new execution context and associated activation object + scope chain is created each time that function is invoked. Since this is a closure, the execution context and associated objects have to wait to be garbage collected.

So you're going to see an increase in memory with each function call until the garbage collector kicks in.

So while the function definition is a literal, the execution most definitely has a cost associated with it.

checker• 13 years ago

If it was an issue, wouldn't it be engine- or browser-specific?

ricardobeat2• 13 years ago

Yes. Firefox had terrible GC pauses until last year [1]. Chrome processes just a portion of the heap at a time to reduce pauses to a minimum [2].

You can minimize GC by being clever about re-using objects, avoiding waste. The top answer shows how GC happens less frequently when using a setTimeout loop instead of setInterval, I don't know the underlying reason but that was always recommended before requestAnimationFrame came along.

These days you should use requestAnimationFrame whenever available - it allows the browser to plan ahead for a smoother framerate, adjust rate to the device capabilities, and save resources when it's in the background.

[1] http://phoboslab.org/log/2011/08/are-we-fast-yet [2] https://developers.google.com/v8/design#garb_coll

malux85• 13 years ago

tl;dr : "No"

ajanuary3• 13 years ago

From what I can tell the issue stems from the fact that Javascript puts stack frames on the heap, which means the heap will grow with every function call, which they're doing every 50 ms.

Does JavaScript's setInterval() method cause memory leak?

Related Stories