Use setImmediate to fix stack overflows in eachSeries.

[mltucker]

async.eachSeries causes stack overflows for large lists.

process.version; // => "v0.10.4"
var async = require("async");
var _ = require("underscore");

var count = 0;
async.eachSeries(_.range(10000), function(n, callback) {
    count += n;
    callback();
}, function(err) {
    console.log(count);
});

Before:

RangeError: Maximum call stack size exceeded

After:

49995000

[diversario]

Having the same issue with stack overflow.

[boutell]

Whoops, I opened a duplicate issue and sent a duplicate pull request on this, so count me in as well.

[dougwilson]

The module is meant to call async functions. Your

function(n, callback) {
    count += n;
    callback();
}

is not async, thus incompatible.

[caolan]

Yes, making functions asynchronous is left up to the developer

[boutell]

OK, I understand that logic. There's a performance penalty imposed by each call to setImmediate too, so it would be redundant extra slowness for functions that are already asynchronous.