Re: [whatwg/streams] Editorial: remove forAuthorCode and use dictionaries for read() return value (#1045)

This now passes all tests. I inserted a manual microtask for teeing, which I think is reasonable. The tests were not able to detect any other instances of timing changes, and I believe that they're exhaustive. The potentially affected instances of ReadableStreamDefaultReaderRead are:

- Teeing, where the timing change matters (as seen here)
- Piping, where the timing change does not matter, since everything observable is intentionally hidden
- Public APIs, where the result gets wrapped into a promise so there are no timing changes

(ReadableStreamBYOBReaderRead only has public API callers, although in theory piping could use it too.)

I'm happy to port this over to the spec text, once folks have had a chance to make any editorial suggestions.

-- 
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/1045#issuecomment-648388272

Received on Tuesday, 23 June 2020 20:05:22 UTC