- From: Domenic Denicola <notifications@github.com>
- Date: Sun, 04 Aug 2024 22:42:43 -0700
- To: whatwg/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <whatwg/webidl/pull/1397/review/2218042834@github.com>
@domenic approved this pull request. > What tests should I write for this? I don't think we have specific webidl tests in WPT, right? Only tests for features that use the webidl types. This was discussed in https://github.com/whatwg/webidl/pull/1397#issuecomment-2021792761. > @@ -6177,6 +6207,34 @@ sequence is used. Any [=list=] can be implicitly treated as a <code>sequence<|T|></code>, as long as it contains only [=list/items=] that are of type |T|. +<!-- Note: if we ever add synchronous iterable types, we should add a note here about why sequences and iterables are not the same. --> + +<h4 id="idl-async-iterable-type" lt="async iterable" dfn export>Async iterable types — async iterable<|T|></h4> + +The <dfn lt="async iterable type" export>async iterable type</dfn> is a parameterized ```suggestion An <dfn lt="async iterable type" export>async iterable type</dfn> is a parameterized ``` > @@ -8115,6 +8173,162 @@ JavaScript Array values. </div> +<h4 id="js-async-iterable">Async iterable — async iterable<|T|></h4> ```suggestion <h4 id="js-async-iterable">Async iterables — async iterable<|T|></h4> ``` (seems like the JS bindings section is very inconsistent on how its headings are named...) > + 1. Otherwise: + 1. Let |V| be [=?=] <a abstract-op>IteratorValue</a>(|iterResult|). + 1. Let |value| be the result of [=converted to an IDL value|converting=] |V| to an IDL + value of type |iterator|'s [=JS async iterator/type parameter=]. + 1. Return |value|. + +</div> + +<div algorithm> + + To <dfn id="async-iterator-close" export lt="close an async iterator">close</dfn> an + <code><a lt="async iterator">async iterator<<var ignore>T</var>></a></code> |iterator|, + with a reason |reason|: + + 1. Let |iteratorRecord| be |iterator|'s [=JS async iterator/underlying record=]. + 1. Let |iteratorObj| be |iteratorRecord|.\[[Iterator]] ```suggestion 1. Let |iteratorObj| be |iteratorRecord|.\[[Iterator]]. ``` -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/webidl/pull/1397#pullrequestreview-2218042834 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/webidl/pull/1397/review/2218042834@github.com>
Received on Monday, 5 August 2024 05:42:47 UTC