-
Notifications
You must be signed in to change notification settings - Fork 27.1k
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.
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
Remove unneeded analyzer ignores #7109
Conversation
We don't need these anymore.
@@ -209,7 +209,6 @@ class ListItem extends StatelessWidget { | |||
height: itemHeight, | |||
padding: const EdgeInsets.symmetric(horizontal: 16.0), | |||
child: new Row( | |||
crossAxisAlignment: CrossAxisAlignment.center, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was this an intentional change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is the default value.
@@ -142,9 +142,9 @@ class AssetImage extends AssetBundleImageProvider { | |||
if (json == null) | |||
return null; | |||
// TODO(ianh): JSON decoding really shouldn't be on the main thread. | |||
final Map<dynamic, dynamic> parsedManifest = JSON.decode(json); | |||
final Map<String, List<String>> parsedManifest = JSON.decode(json); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's not right. JSON.decode will not return a Map<String, List<String>>
. It has a return type of dynamic
and (if the JSON is what we expect) return a Map<String, dynamic>
that happens to only contain List<String>
s.
// TODO(ianh): convert that data structure to the right types. | ||
return new SynchronousFuture<Map<dynamic, dynamic>>(parsedManifest); // ignore: return_of_invalid_type, https://github.com/flutter/flutter/issues/5771 | ||
return new SynchronousFuture<Map<String, List<String>>>(parsedManifest); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should fail in strong mode, since it's an invalid cast, as far as I can tell.
GlobalKey globalKey = key; | ||
return globalKey.currentState; // ignore: return_of_invalid_type, https://github.com/flutter/flutter/issues/5771 | ||
GlobalKey<T> globalKey = key; | ||
return globalKey.currentState; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huh. I thought I'd tried that and it didn't work for some reason. Weird.
We don't need these anymore.