Re: [whatwg/streams] Is autoAllocateChunkSize permitted to be zero? (#1089)

This regressed in the Web IDL transition. https://streams.spec.whatwg.org/commit-snapshots/cfcd303a38158e4ec3837d1028158c88d505c35d/#set-up-readable-byte-stream-controller-from-underlying-source contains

> If ! IsInteger(autoAllocateChunkSize) is false, or if autoAllocateChunkSize ≤ 0, throw a RangeError exception.

but this was removed in the Web IDL transition because I thought it was no longer necessary. I forgot about 0.

We should add back in a line specifically for 0.

Bonus fix to make at the same time: add `[$ $]` around the IsInteger in https://streams.spec.whatwg.org/commit-snapshots/41a968974c91e05b20738f77d0b2731ec7391e92/#set-up-readable-byte-stream-controller.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/issues/1089#issuecomment-736006070

Received on Monday, 30 November 2020 19:55:19 UTC