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
Add async value iterators and async iterator arguments #808
Conversation
d424411
to
719a360
Compare
719a360
to
8cd2cd6
Compare
This comment has been minimized.
This comment has been minimized.
8cd2cd6
to
bba882b
Compare
Rebased on master, made LL(1), and ready for review! |
Ping on reviews? |
async iterable<value_type>; | ||
async iterable<value_type>(/* arguments... */); | ||
async iterable<key_type, value_type>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably also be listed, given that it's valid:
async iterable<key_type, value_type>(/* arguments... */);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you and sorry for the terrible lag.
index.bs
Outdated
@@ -12655,8 +12725,16 @@ The \[[Prototype]] [=internal slot=] of an [=asynchronous iterator prototype obj | |||
1. Set |object|'s [=default asynchronous iterator object/is finished=] to true. | |||
1. Return [=!=] [$CreateIterResultObject$](<emu-val>undefined</emu-val>, | |||
<emu-val>true</emu-val>). | |||
1. Otherwise: | |||
1. Otherwise, if the corresponding [=asynchronously iterable declaration=] is a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have interface
defined in step 1,
- Let interface be the interface for which the asynchronous iterator prototype object exists.
could we use something like if interface has a pair asynchronously iterable declaration
instead?
index.bs
Outdated
1. Return the [=iterator result=] for |next| and |kind|. | ||
1. Otherwise: | ||
1. Assert: the corresponding [=asynchronously iterable declaration=] is a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And here.
Thanks for the review! Code review comments addressed, plus removed an extra clause that I saw in my own re-review. |
Closes #800.
Preview | Diff