Define ReadableByteStream[Reader].

yutakahirano / fetch-with-streams

Fetch API integrated with Streams.

66 stars 8 forks source link

Closed yutakahirano closed 9 years ago

yutakahirano commented 9 years ago

@wanderview, @domenic, @tyoshino, @kenjibaheux Can I define ReadableByteStream[Reader] here as a workaround for the concern?

yutakahirano commented 9 years ago

Justification: We are sure that a ReadableByteStream can be used as a ReadableStream from the user-side.

tyoshino commented 9 years ago

lgtm

domenic commented 9 years ago

LGTM too. @wanderview does this help?

wanderview commented 9 years ago

That does definitely help. Thanks!

I would still personally just use ReadableStream for now, but this is probably adequate if you are ok locking yourself into the superset requirement.