-
Notifications
You must be signed in to change notification settings - Fork 61
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
process.nextTick is slow #86
Comments
hey so I replied in the other issue but the gist is that there are some reasons for not putting that fancy fast things in this default library but i have a different library that it might make sense to use. |
What about try to use queueMicrotask and use the old one behavior as a fallback? |
Or if we use https://github.com/calvinmetcalf/immediate we can add support for window.queueMicrotask there :) |
the latter is probably a better option, this library get's included in a lot of bundles so the aim is to keep is small |
It's true is everywhere 😄 and I understand your point but if you don't mind I'm going to do a PR with the changes that I was thinking and you can tell me if it's too much. |
We've just run into this. It's particularly bad if your test suite relies on I've tried monkey-patching with the proposed |
setTimeout
is clamped down to 4ms and even more when tab is hidden.I saw some old PR attempt at using Observer PostMessage immidate and other but you still use
setTimeout
and that came back and bite us when transferring data webtorrent/webtorrent#1568Could you look into using something faster then setTimeout?
The text was updated successfully, but these errors were encountered: