Re: [whatwg/streams] Add ReadableStreamBYOBReader.read(view, { min }) (#1145)

@MattiasBuelens commented on this pull request.



>    undefined releaseLock();
 };
 ReadableStreamBYOBReader includes ReadableStreamGenericReader;
+
+dictionary ReadableStreamBYOBReaderReadOptions {
+  [EnforceRange] unsigned long long min;

Hmm, good point. 🤔 

We currently have slightly different logic between the `min` and non-`min` cases. It would be much nicer to convert all non-`min` cases to have a default `min`, and then just have `min` everywhere.

I'll look into it.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1145#discussion_r1224441576
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/streams/pull/1145/review/1472397581@github.com>

Received on Friday, 9 June 2023 15:16:49 UTC