- From: Takeshi Yoshino <notifications@github.com>
- Date: Thu, 18 Feb 2016 03:41:10 -0800
- To: whatwg/streams <streams@noreply.github.com>
- Message-ID: <whatwg/streams/pull/430@github.com>
This commit includes the following design changes:
- sync the spec text with the new reference implementation
- #423
- add byobRequest getter to the controller
- port the asserts in the non-byte source version to the byte source version
- remove auto release feature from the byte source version
- rename Byob to BYOB
- move TransferArrayBuffer to helpers.js
- make the default highWaterMark of the byte source version to 0
- port the functionality that the start method can delay pulling by returning a pending Promise to the byte source version
- add `[[disturbed]]` feature to the byte source version
- port highWaterMark mechanism to ReadableByteStreamController
- merge ReadableStream and ReadableByteStream
- now it behaves differently based on whether the given source has pullInto or not
- bunch of renaming for merge
- merge ReadableStreamReader and ReadableByteStreamReader
- #424
- make controller.close() and controller.enqueue() fail when the stream is not in the readable state
- make controller.enqueue() throw a predefined TypeError, not the stored error
You can view, comment on, or merge this pull request online at:
https://github.com/whatwg/streams/pull/430
-- Commit Summary --
* Merge ReadableByteStream into ReadableStream
-- File Changes --
M reference-implementation/lib/helpers.js (24)
M reference-implementation/lib/queue-with-sizes.js (3)
M reference-implementation/lib/readable-stream.js (1554)
M reference-implementation/run-tests.js (2)
M reference-implementation/test/pipe-to-options.js (68)
M reference-implementation/test/readable-byte-stream.js (1294)
M reference-implementation/test/readable-stream-templated.js (38)
M reference-implementation/web-platform-tests/readable-streams/bad-underlying-sources.js (18)
M reference-implementation/web-platform-tests/readable-streams/cancel.js (1)
M reference-implementation/web-platform-tests/readable-streams/general.js (52)
-- Patch Links --
https://github.com/whatwg/streams/pull/430.patch
https://github.com/whatwg/streams/pull/430.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/430
Received on Thursday, 18 February 2016 11:41:43 UTC