Skip to content
This repository has been archived by the owner on Feb 24, 2025. It is now read-only.

Prepare for upcoming change to HttpRequest and HttpClientResponse #254

Merged
merged 1 commit into from
Jun 26, 2019

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

Copy link
Collaborator

@srawlins srawlins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Todd!

Copy link
Collaborator

@srawlins srawlins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just need a dartfmt :)

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
@tvolkert
Copy link
Contributor Author

done!

@srawlins
Copy link
Collaborator

Thanks again, Todd

@srawlins srawlins merged commit 38b0fa5 into dart-archive:master Jun 26, 2019
@tvolkert tvolkert deleted the http branch June 26, 2019 14:40
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 9, 2024
…rt-archive/markdown#254)

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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants