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

Unified Diff: editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/parser/Parser.java

Issue 189803004: Translate private Java members to private Dart members. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Tweaks Created 6 years, 9 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/parser/Parser.java
diff --git a/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/parser/Parser.java b/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/parser/Parser.java
index 01ef6ec2ec2f9d06775d1d9c7c7a070a9865c6ed..e48896c3e0423ceb0a3e7075f1ca74c5666b6d68 100644
--- a/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/parser/Parser.java
+++ b/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/parser/Parser.java
@@ -775,6 +775,23 @@ public class Parser {
return new ConstructorName(type, period, name);
}
+// /**
+// * If the current token is an identifier matching the given identifier, return it after advancing
+// * to the next token. Otherwise report an error and return the current token without advancing.
+// *
+// * @param identifier the identifier that is expected
+// * @return the token that matched the given type
+// */
+// private Token expect(String identifier) {
+// if (matches(identifier)) {
+// return getAndAdvance();
+// }
+// // Remove uses of this method in favor of matches?
+// // Pass in the error code to use to report the error?
+// reportError(ParserErrorCode.EXPECTED_TOKEN, identifier);
+// return currentToken;
+// }
+
/**
* Parse an expression that does not contain any cascades.
*
@@ -818,23 +835,6 @@ public class Parser {
return expression;
}
-// /**
-// * If the current token is an identifier matching the given identifier, return it after advancing
-// * to the next token. Otherwise report an error and return the current token without advancing.
-// *
-// * @param identifier the identifier that is expected
-// * @return the token that matched the given type
-// */
-// private Token expect(String identifier) {
-// if (matches(identifier)) {
-// return getAndAdvance();
-// }
-// // Remove uses of this method in favor of matches?
-// // Pass in the error code to use to report the error?
-// reportError(ParserErrorCode.EXPECTED_TOKEN, identifier);
-// return currentToken;
-// }
-
/**
* Parse an expression that does not contain any cascades.
*
@@ -867,22 +867,6 @@ public class Parser {
return expression;
}
- /**
- * Parse a class extends clause.
- *
- * <pre>
- * classExtendsClause ::=
- * 'extends' type
- * </pre>
- *
- * @return the class extends clause that was parsed
- */
- protected ExtendsClause parseExtendsClause() {
- Token keyword = expectKeyword(Keyword.EXTENDS);
- TypeName superclass = parseTypeName();
- return new ExtendsClause(keyword, superclass);
- }
-
// /**
// * If the current token is an identifier matching the given identifier, return it after advancing
// * to the next token. Otherwise report an error and return the current token without advancing.
@@ -901,6 +885,22 @@ public class Parser {
// }
/**
+ * Parse a class extends clause.
+ *
+ * <pre>
+ * classExtendsClause ::=
+ * 'extends' type
+ * </pre>
+ *
+ * @return the class extends clause that was parsed
+ */
+ protected ExtendsClause parseExtendsClause() {
+ Token keyword = expectKeyword(Keyword.EXTENDS);
+ TypeName superclass = parseTypeName();
+ return new ExtendsClause(keyword, superclass);
+ }
+
+ /**
* Parse a list of formal parameters.
*
* <pre>
@@ -1378,18 +1378,18 @@ public class Parser {
}
/**
- * Parse a list of type arguments.
- *
- * <pre>
- * typeArguments ::=
- * '<' typeList '>'
- *
- * typeList ::=
- * type (',' type)*
- * </pre>
- *
- * @return the type argument list that was parsed
- */
+ * Parse a list of type arguments.
+ *
+ * <pre>
+ * typeArguments ::=
+ * '<' typeList '>'
+ *
+ * typeList ::=
+ * type (',' type)*
+ * </pre>
+ *
+ * @return the type argument list that was parsed
+ */
protected TypeArgumentList parseTypeArgumentList() {
Token leftBracket = expect(TokenType.LT);
List<TypeName> arguments = new ArrayList<TypeName>();
@@ -1461,15 +1461,15 @@ public class Parser {
}
/**
- * Parse a list of type parameters.
- *
- * <pre>
- * typeParameterList ::=
- * '<' typeParameter (',' typeParameter)* '>'
- * </pre>
- *
- * @return the list of type parameters that were parsed
- */
+ * Parse a list of type parameters.
+ *
+ * <pre>
+ * typeParameterList ::=
+ * '<' typeParameter (',' typeParameter)* '>'
+ * </pre>
+ *
+ * @return the list of type parameters that were parsed
+ */
protected TypeParameterList parseTypeParameterList() {
Token leftBracket = expect(TokenType.LT);
List<TypeParameter> typeParameters = new ArrayList<TypeParameter>();

Powered by Google App Engine
This is Rietveld 408576698