- From: Domenic Denicola <notifications@github.com>
- Date: Mon, 30 Nov 2020 11:55:07 -0800
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 30 November 2020 19:55:19 UTC
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