Re: [whatwg/streams] ReadableStream constructor: standard text and reference implementation doesn't match (#917)

> Having said that, I haven't written a C++ implementation and I don't know how big of a problem this is in practice. @tschneidereit, do you have any input?

I'm entirely fine with changing the order of operations here - pretty much in whichever way works best for a JS implementation or has more internal consistency. I agree with @domenic that the order shouldn't be implementation-defined: that seems like an entirely unnecessary source of potential subtle bugs.

> implementations that would prefer to use Web IDL can be content to fail the ordering tests.

If we have implementations that already know that they'd go this route, then we might as well admit that we're leaving the order implementation-defined :/

-- 
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/issues/917#issuecomment-378697334

Received on Wednesday, 4 April 2018 18:22:51 UTC