Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

async.each should not crash with undefined entry #715

Closed
ajcrites opened this issue Feb 12, 2015 · 1 comment
Closed

async.each should not crash with undefined entry #715

ajcrites opened this issue Feb 12, 2015 · 1 comment

Comments

@ajcrites
Copy link

var foo;
async.each(foo);

This will yield an error:

node_modules/async/lib/async.js:120
    if (!arr.length) {

I think that if arr is undefined it should be treated like an empty array or should trigger an error in the callback. I have a strong preference for the former, though.

@aearly
Copy link
Collaborator

aearly commented Feb 12, 2015

Related: #667

@aearly aearly closed this as completed May 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants