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

Rewrite async page+spread operators for native promises #76

Closed
vitaly-t opened this issue Dec 21, 2021 · 1 comment
Closed

Rewrite async page+spread operators for native promises #76

vitaly-t opened this issue Dec 21, 2021 · 1 comment
Labels
enhancement New feature or request
Milestone

Comments

@vitaly-t
Copy link
Owner

vitaly-t commented Dec 21, 2021

Async operators page and spread should be easy to replace with promise-only implementation, for maximum performance.

Review the rest of async syntax.

@vitaly-t vitaly-t added this to the v1.5 milestone Dec 21, 2021
@vitaly-t vitaly-t added the enhancement New feature or request label Dec 21, 2021
@vitaly-t
Copy link
Owner Author

vitaly-t commented Dec 22, 2021

This has been done and released in version 1.4.8

Remaining async operators that still use async/await syntax:

  • concat
  • defaultEmpty
  • repeat
  • split

That's 4 out of 35 total operators, not bad.

Perhaps repeat is the only one worth revisiting later, while the rest are just too big, and rewriting them into native promises won't bring much benefit, if any.

@vitaly-t vitaly-t changed the title Replace async page+spread operators with promise implementation Rewrite async page+spread operators for native promises Dec 22, 2021
@vitaly-t vitaly-t mentioned this issue Oct 13, 2022
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant