Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(904)

Unified Diff: editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/resolver/ElementResolver.java

Issue 15004020: Report StaticWarningCode.IMPORT_DUPLICATED_LIBRARY_NAME (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Use multiline test sources Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/resolver/ElementResolver.java
diff --git a/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/resolver/ElementResolver.java b/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/resolver/ElementResolver.java
index 5532cd276ca8f762f5b727c8f61b951270f69d37..c72c8a64ea552738d25d63e5f4f44f08e8ce5dec 100644
--- a/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/resolver/ElementResolver.java
+++ b/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/resolver/ElementResolver.java
@@ -554,7 +554,11 @@ public class ElementResolver extends SimpleASTVisitor<Void> {
// The element is null when the URI is invalid
// TODO(brianwilkerson) Figure out whether the element can ever be something other than an
// ImportElement
- resolveCombinators(((ImportElement) element).getImportedLibrary(), node.getCombinators());
+ ImportElement importElement = (ImportElement) element;
+ LibraryElement library = importElement.getImportedLibrary();
+ if (library != null) {
+ resolveCombinators(library, node.getCombinators());
+ }
}
return null;
}

Powered by Google App Engine
This is Rietveld 408576698