Skip to content
This repository has been archived by the owner on Oct 17, 2024. It is now read-only.

Two small strong-mode/warning fixes #20

Merged
merged 1 commit into from
Jan 24, 2017

Conversation

matanlurey
Copy link
Contributor

Removing an unused import and fixing a dynamic inference.

@matanlurey matanlurey requested a review from nex3 January 20, 2017 19:58
@@ -101,5 +100,5 @@ class _TransformedSubscription<T> implements StreamSubscription<T> {
}

Future/*<E>*/ asFuture/*<E>*/([/*=E*/ futureValue]) =>
_inner?.asFuture(futureValue) ?? new Completer().future;
_inner?.asFuture(futureValue) ?? new Completer/*<E>*/().future;
Copy link
Contributor

Choose a reason for hiding this comment

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

@jmesserly Shouldn't this be inferred?

Choose a reason for hiding this comment

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

Hmmm. Not sure, I know we have inference for ??. Do you know what the error message was?

Copy link

@jmesserly jmesserly Jan 25, 2017

Choose a reason for hiding this comment

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

oooooooooh ... NVM, this is the bug: dart-lang/sdk#27114

Copy link
Contributor

Choose a reason for hiding this comment

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

Unsafe implicit cast from 'Future<dynamic>' to 'Future<E>'.

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