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.

Sign up for GitHub

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

Jump to bottom

async.nextTick should be defined once #8

Closed
kpozin opened this issue Dec 16, 2010 · 1 comment
Closed

async.nextTick should be defined once #8

kpozin opened this issue Dec 16, 2010 · 1 comment

Comments

@kpozin
Copy link
Contributor

kpozin commented Dec 16, 2010

async.nextTick checks for the existence of process.nextTick on every call, which is somewhat inefficient considering how often it's called. Why not define it once, since the execution environment will never change between node.js and a browser?

a3ff995
@caolan
Copy link
Owner

caolan commented Dec 19, 2010

Good idea, thanks for your patch :)

I've merged it into the main branch.

aearly pushed a commit that referenced this issue Jun 30, 2016
@hargasinski
jsdoc: improved tt-menu styling, closes #8, @megawac
01b0a19
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@caolan @kpozin