Re: [whatwg/streams] Fix queue total size tracking logic (#661)

> I guess that is a bit weird that sometimes when the queue is empty during the course of operation desiredSize will be nonzero because of floating-point shenanigans, and sometimes it will be reset to 0. We could either accept this, or manually make DequeueValue reset the total size to 0 if the queue becomes empty.

Or we could clamp the queue size to >= 0 in DequeueValue, ensuring that we'll always get predictable results. I might be missing something that makes this undesirable.

Apart from that point, I can't see anything amiss here, looks great!

-- 
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/661#issuecomment-279177931

Received on Saturday, 11 February 2017 21:44:13 UTC