Re: [heycam/webidl] Add async value iterators and async iterator arguments (#808)

@mkruisselbrink commented on this pull request.



> @@ -4295,28 +4295,41 @@ An [=interface=] can be declared to be asynchronously iterable by using an
 (matching <emu-nt><a href="#prod-AsyncIterable">AsyncIterable</a></emu-nt>) in the body of the
 [=interface=].
 
-<pre highlight="webidl" class="syntax">
+<!-- TODO: add highlight="idl" after idlparser gets updated -->

highlight="idl" or should that be highlight="webidl" like before?

>  Note: In the ECMAScript language binding, an interface that is asynchronously iterable will have
-<code class="idl">entries</code>, <code class="idl">keys</code>, <code class="idl">values</code>,
-and {{@@asyncIterator}} properties on its [=interface prototype object=].
+{{@@asyncIterator}} and <code class="idl">values</code> properties on its

perhaps I'm just not familiar enough with the subject, but it would help me understand things better if some note explained that the optional arguments for the async iterable definition end up as arguments you can pass to the values method.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/pull/808#pullrequestreview-382179053