Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"At this point in time the division of work betw..." #741

Closed
ricea opened this issue Apr 28, 2017 · 1 comment
Closed

"At this point in time the division of work betw..." #741

ricea opened this issue Apr 28, 2017 · 1 comment
Labels
editorial Changes that do not affect how the standard is understood. writable streams

Comments

@ricea
Copy link
Collaborator

ricea commented Apr 28, 2017

https://streams.spec.whatwg.org/commit-snapshots/3193db4f3fa34c52d726055ab99a623f66c4a623#ws-abstract-ops-used-by-controllers

At this point in time the division of work between the stream and its controller may seems somewhat arbitrary, but centralizing much of the logic in the controller is a useful structure for the future.

This sentence is maybe overly ambiguous? I feel we've actually been quite careful in designing this to enable byte streams, but that doesn't come across at all.

Based on w3ctag/design-reviews#163 (comment) by @dbaron.
@ricea ricea added editorial Changes that do not affect how the standard is understood. writable streams labels Apr 28, 2017
@tyoshino
Copy link
Member

Shall we just note concretely that this is for extending WS for a byte stream variant?

domenic added a commit that referenced this issue Sep 15, 2017
@domenic
Don't be dismissive about the controller/stream division
20c1a3f 
Closes #741.
@domenic domenic mentioned this issue Sep 15, 2017
Merged
ricea pushed a commit that referenced this issue Sep 15, 2017
@domenic @ricea
Don't be dismissive about the controller/stream division (#797)
87cdc05 
Closes #741.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
editorial Changes that do not affect how the standard is understood. writable streams
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tyoshino @ricea