- 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