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

Unified Diff: compiler/java/com/google/dart/compiler/resolver/CompileTimeConstantResolver.java

Issue 10582013: Issue 1655. Check initializer values of const constructors (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 6 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/resolver/CompileTimeConstantResolver.java
diff --git a/compiler/java/com/google/dart/compiler/resolver/CompileTimeConstantResolver.java b/compiler/java/com/google/dart/compiler/resolver/CompileTimeConstantResolver.java
index 3f82debc0ed8c7a8e535a3daaef16d1e389e4577..96c866341384794ec3c66b64a682cc99935860ad 100644
--- a/compiler/java/com/google/dart/compiler/resolver/CompileTimeConstantResolver.java
+++ b/compiler/java/com/google/dart/compiler/resolver/CompileTimeConstantResolver.java
@@ -181,6 +181,14 @@ public class CompileTimeConstantResolver {
resolveConstantExpression(node.getValue());
return null;
}
+
+ @Override
+ public Element visitInitializer(DartInitializer node) {
+ if (node.getInitializerName() != null) {
+ resolveConstantExpression(node.getValue());
+ }
+ return super.visitInitializer(node);
+ }
@Override
public Element visitMethodDefinition(DartMethodDefinition node) {

Powered by Google App Engine
This is Rietveld 408576698