- From: isonmad <notifications@github.com>
- Date: Fri, 13 Jan 2017 17:48:46 -0800
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Saturday, 14 January 2017 01:49:27 UTC
It was special-casing the first read request, and adding a second one would assert ever since it was first merged in https://github.com/whatwg/streams/pull/430#issuecomment-195370054. ```js var rs = new ReadableStream({type:'bytes', autoAllocateChunkSize:1}); var r = rs.getReader(); var p1 = r.read(); var p2 = r.read(); // assertion failure ``` Also combined two redundant nested if statements, since `totalQueuedBytes > 0` implies `ReadableStreamGetNumReadRequests(stream) === 0`, AFAICT. You can view, comment on, or merge this pull request online at: https://github.com/whatwg/streams/pull/651 -- Commit Summary -- * fix autoAllocateChunkSize * fixup whitespace indent -- File Changes -- M index.bs (38) M reference-implementation/lib/readable-stream.js (64) -- Patch Links -- https://github.com/whatwg/streams/pull/651.patch https://github.com/whatwg/streams/pull/651.diff -- 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/pull/651
Received on Saturday, 14 January 2017 01:49:27 UTC