RangeError when using queue with 5000+ tasks.

[LiamKarlMitchell]

I have made a pull request with a test that fails.
#509

RangeError Maximum call stack size exceeded

It is basicaly this

var q = async.queue(function(task,cb) { cb(); })
q.pause();

for (var i=0; i< 5000; i++) {
  q.push({ index: i });
}

q.resume();

When cb is called there can be RangeError's
I don't know enough to figure out why though.

[LiamKarlMitchell]

eachLimit and eachSeries also has this problem.

[caolan]

You need to call cb() asynchronously. Try setImmediate(cb) instead.

[LiamKarlMitchell]

Ah my mistake thankyou @caolan
Just wondering couldn't the call back be internally handled to call async to avoid a mistake like this?
Or is there a good reason that it is required?

Cheers,

[caolan]

Making your sync functions magically async would just mask the problem, so I prefer not to add it into the library.

[bishopZ]

Btw, Frame.js v2 resolved this issue. https://github.com/bishopZ/Frame.js