Skip to content

Conversation

tvolkert
Copy link
Contributor

An upcoming change to the Dart SDK will change HttpRequest and
HttpClientResponse from implementing Stream<List<int>> to
implementing Stream<Uint8List>.

This forwards-compatible change prepares for that SDK breaking
change by casting the Stream to List<int> before transforming
it.

dart-lang/sdk#36900

An upcoming change to the Dart SDK will change `HttpRequest` and
`HttpClientResponse` from implementing `Stream<List<int>>` to
implementing `Stream<Uint8List>`.

This forwards-compatible change prepares for that SDK breaking
change by casting the Stream to `List<int>` before transforming
it.

dart-lang/sdk#36900
@MichaelMarner
Copy link
Owner

Hi Todd, thanks for the contribution and for going into so much detail finding other projects that would be affected by the Dart API change (dart-lang/sdk#36900).

I'm not sure what happened with Travis, because the build and tests all passed. Happy to merge.

@MichaelMarner MichaelMarner merged commit 42a22e9 into MichaelMarner:master Jun 27, 2019
@tvolkert tvolkert deleted the http branch June 27, 2019 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants