Re: [whatwg/streams] Various fixes for readable byte streams (#1123)

> I think that after you successfully called controller.byobRequest.respond() or .respondWithNewView(), the BYOB request should be invalidated and its buffer should be transferred. You should no longer be able to modify the old value of controller.byobRequest.view

This makes sense to me.

> Hmm, enqueue() also invalidates the BYOB request but doesn't always detach the previous request's buffer either. Maybe this was intentional?

I don't think it was intentional... unless we wrote tests exercising it. Probably tightening it up is safer until/unless someone discovers a use case.

---

This PR is getting rather large, but all good fixes. I have no problem doing them all together, but would you be able to update the OP with a running summary of the changes, in the form of what an eventual commit message would look like?

-- 
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/1123#issuecomment-825235305

Received on Thursday, 22 April 2021 23:00:48 UTC