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

Unified Diff: compiler/java/com/google/dart/compiler/ast/DartField.java

Issue 9148026: Recompile unit with potential conflict/dependency on some top-level symbol. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix for compiling corelib, so NPE in TreeShaker Created 8 years, 11 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: compiler/java/com/google/dart/compiler/ast/DartField.java
diff --git a/compiler/java/com/google/dart/compiler/ast/DartField.java b/compiler/java/com/google/dart/compiler/ast/DartField.java
index 4a2c587fab491aecf9d784e50ecbde890da9cf31..1718ca80b14571b3e488448846f6461893f3434d 100644
--- a/compiler/java/com/google/dart/compiler/ast/DartField.java
+++ b/compiler/java/com/google/dart/compiler/ast/DartField.java
@@ -77,19 +77,4 @@ public class DartField extends DartClassMember<DartIdentifier> {
public <R> R accept(DartPlainVisitor<R> visitor) {
return visitor.visitField(this);
}
-
- @Override
- public int computeHash() {
- // TODO(jgw): Remove this altogether in fixing b/5324113.
-
- // DartField doesn't include the type-node, so we directly return the hash of its type, which
- // is all that matters for the purposes of dependency-tracking.
- DartFieldDefinition def = (DartFieldDefinition) getParent();
- DartTypeNode typeNode = def.getTypeNode();
- if (typeNode == null) {
- // Use 0 to represent an untyped field.
- return 0;
- }
- return typeNode.computeHash();
- }
}

Powered by Google App Engine
This is Rietveld 408576698