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

Unified Diff: compiler/java/com/google/dart/compiler/parser/ParserErrorCode.java

Issue 10690074: Migrates some resolver errors to parser stage (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/parser/ParserErrorCode.java
diff --git a/compiler/java/com/google/dart/compiler/parser/ParserErrorCode.java b/compiler/java/com/google/dart/compiler/parser/ParserErrorCode.java
index 0e66151c4574b8fdf4ab52e4f84fc1b18dc0213b..583fc2941598f3f8ead1de7084bd6f52a8f8f4e1 100644
--- a/compiler/java/com/google/dart/compiler/parser/ParserErrorCode.java
+++ b/compiler/java/com/google/dart/compiler/parser/ParserErrorCode.java
@@ -34,9 +34,10 @@ public enum ParserErrorCode implements ErrorCode {
DEFAULT_POSITIONAL_PARAMETER("Positional parameters cannot have default values"),
DEPRECATED_USE_OF_FACTORY_KEYWORD("Deprecated use of the 'factory' keyword: use 'default' instead"),
DIRECTIVE_OUT_OF_ORDER("Directive out of order"),
- // TODO(zundel): error message needs JUnit test
DISALLOWED_ABSTRACT_KEYWORD("Abstract keyword not allowed here"),
+ // TODO(zundel): error message needs JUnit test
DISALLOWED_FACTORY_KEYWORD("Factory keyword not allowed here"),
+ DISALLOWED_STATIC_KEYWORD("Static keyword not allowed here"),
// TODO(zundel): error message needs JUnit test
EXPECTED_ARRAY_OR_MAP_LITERAL("Expected array or map literal"),
// TODO(zundel): error message needs JUnit test
@@ -50,6 +51,7 @@ public enum ParserErrorCode implements ErrorCode {
// TODO(zundel): error message needs JUnit test
EXPECTED_EXPRESSION_AFTER_COMMA("Expected expression after comma"),
EXPECTED_EXTENDS("Expected 'extends'"),
+ EXPECTED_FUNCTION_STATEMENT_BODY("Expected function statment body"),
// TODO(zundel): error message needs JUnit test
EXPECTED_IDENTIFIER("Expected identifier"),
// TODO(zundel): error message needs JUnit test
@@ -74,6 +76,7 @@ public enum ParserErrorCode implements ErrorCode {
FACTORY_CANNOT_BE_ABSTRACT("A factory cannot be abstract"),
// TODO(zundel): error message needs JUnit test
FACTORY_CANNOT_BE_STATIC("A factory cannot be static"),
+ FACTORY_CANNOT_BE_TOPLEVEL("A factory cannot be toplevel, it must be a member of a class"),
// TODO(zundel): error message needs JUnit test
FACTORY_MEMBER_IN_INTERFACE("Factory members are not allowed in interfaces"),
FINAL_IS_NOT_ALLOWED_ON_A_METHOD_DEFINITION("'final' is not allowed on a method definition"),
@@ -117,6 +120,7 @@ public enum ParserErrorCode implements ErrorCode {
REDIRECTING_CONSTRUCTOR_ITSELF("Redirecting constructor cannot have initializers"),
REDIRECTING_CONSTRUCTOR_MULTIPLE("Multiple redirecting constructor invocations"),
REDIRECTING_CONSTRUCTOR_OTHER("Redirecting constructor cannot have initializers"),
+ STATIC_MEMBERS_CANNOT_BE_ABSTRACT("Static members cannot be abstract"),
SUPER_CONSTRUCTOR_MULTIPLE("'super' must be called only once in the initialization list"),
SUPER_CANNOT_BE_USED_AS_THE_SECOND_OPERAND(
"'super' cannot be used as the second operand in a binary expression."),

Powered by Google App Engine
This is Rietveld 408576698