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

Accommodating future addition of byte types to WritableStream and TransformStream makes this messier than I realised. I'd like to list out my priorities, for my own benefit as much as anything. From high importance to low importance:

1. Avoid duplication. In particular, this means that size needs to be looked up separately from validating it, since the validation depends on the type of the stream, but the lookup is the same in both cases.
2. Try to avoid having to change the order again when we add byte streams.
3. Perform lookups close to the point of use.
4. Perform validation soon after lookup.
5. Be consistent. The ordering should match between the different types where it doesn't contradict the above.
6. Be explicit.

I suppose the alternative would be to not change anything and just test for the current order. But personally I'd rather take the opportunity to make it make sense.

-- 
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-379650790