Skip to content
This repository was archived by the owner on Aug 28, 2024. It is now read-only.

Fix warnings related to fuzzy arrows. #209

Merged
merged 1 commit into from
Nov 9, 2017
Merged

Fix warnings related to fuzzy arrows. #209

merged 1 commit into from
Nov 9, 2017

Conversation

munificent
Copy link
Contributor

These will become an arrow in Dart 2.0. See this bug for more context:

dart-lang/sdk#29630

I also fixed a couple of unused variable warnings while I was at it.

I bumped the version number in the pubspec. This is technically a
potentially breaking change because the signatures of createHitmap() and
mergeHitmaps() are more precise now. The only code I could find calling
that was in flutter_tools and that seems to be happy with the change.
If you'd rather I keep the old version, let me know.

These will become an arrow in Dart 2.0. See this bug for more context:

  dart-lang/sdk#29630

I also fixed a couple of unused variable warnings while I was at it.

I bumped the version number in the pubspec. This is technically a
potentially breaking change because the signatures of createHitmap() and
mergeHitmaps() are more precise now. The only code I could find calling
that was in flutter_tools and that seems to be happy with the change.
If you'd rather I keep the old version, let me know.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 80.851% when pulling 585923e on fuzzy-arrows into 53daab4 on master.

@cbracken
Copy link
Contributor

cbracken commented Nov 9, 2017

Thank you, good sir!

@cbracken cbracken merged commit bf9211f into master Nov 9, 2017
@cbracken cbracken deleted the fuzzy-arrows branch November 9, 2017 01:27
cbracken added a commit to cbracken/coverage that referenced this pull request Dec 14, 2017
cbracken added a commit that referenced this pull request Dec 14, 2017
mosuem pushed a commit to dart-lang/tools that referenced this pull request Aug 28, 2024
These will become an arrow in Dart 2.0. See this bug for more context:

  dart-lang/sdk#29630

I also fixed a couple of unused variable warnings while I was at it.

I bumped the version number in the pubspec. This is technically a
potentially breaking change because the signatures of createHitmap() and
mergeHitmaps() are more precise now. The only code I could find calling
that was in flutter_tools and that seems to be happy with the change.
If you'd rather I keep the old version, let me know.
mosuem pushed a commit to dart-lang/tools that referenced this pull request Aug 28, 2024
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.

4 participants