Re: [streams] Convert readable stream pull() to promise-returning (#272)

> Even if the read() calls drain the queue, pull() won't be called until the next microtask is run. Are you aware of this? I'm fine with this but want to confirm.

Yeah. I tried to address this with the commit message. If the underlying source has more chunks available synchronously, it should just enqueue them. Even if the backpressure signal says the queue is full, it should use that to stop any flow, but still enqueue any chunks it already has. Whereas, if the underlying source can't get any more chunks without an asynchronous delay, there will be no problem anyway. Make sense?

Going to merge but happy to make any updates if e.g. I am misunderstanding https://github.com/whatwg/streams/pull/272#discussion_r24150104

---
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/272#issuecomment-73095068

Received on Thursday, 5 February 2015 18:05:34 UTC