DartC parser stress test (2)

compiler/java/com/google/dart/compiler/parser/DartParser.java

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 package com.google.dart.compiler.parser;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.io.CharStreams;
 import com.google.dart.compiler.DartCompilationError;
 import com.google.dart.compiler.DartCompilerErrorCode;
@@ skipping 165 matching lines @@
 
   public DartParser(ParserContext ctx, boolean isDietParse, Set<String> prefixes) {
     super(ctx);
     this.isDietParse = isDietParse;
     this.prefixes = prefixes;
   }
 
   private DartParser(Source source, DartCompilerListener listener) throws IOException {
     this(source, source.getSourceReader(), listener);
   }
-  
+
   private DartParser(Source source,
                     Reader sourceReader,
                     DartCompilerListener listener) throws IOException {
     this(new DartScannerParserContext(source, read(sourceReader), listener));
   }
 
   public static DartParser getSourceParser(Source source, DartCompilerListener listener)
       throws IOException {
     return new DartParser(source, listener);
   }
@@ skipping 70 matching lines @@
       if (node != null) {
         unit.addTopLevelNode(node);
       }
     }
     expect(Token.EOS);
     return done(unit);
   }
 
   /**
    * A version of the parser which only parses the directives of a library.
-   * 
+   *
    * TODO(jbrosenberg): consider parsing the whole file here, in order to avoid
    * duplicate work.  Probably requires removing use of LibraryUnit's, etc.
    * Also, this minimal parse does have benefit in the incremental compilation
    * case.
    */
   public LibraryUnit preProcessLibraryDirectives(LibrarySource source) {
     beginCompilationUnit();
     LibraryUnit libUnit = new LibraryUnit(source);
     if (peek(0) == Token.LIBRARY) {
       beginLibraryDirective();
@@ skipping 341 matching lines @@
     DartIdentifier name = parseIdentifier();
     List<DartTypeParameter> typeParameters = parseTypeParametersOpt();
     List<DartParameter> params = parseFormalParameterList();
     expect(Token.SEMICOLON);
 
     return done(new DartFunctionTypeAlias(name, returnType, params, typeParameters));
   }
 
   /**
    * Parse a field or method, which may be inside a class or at the top level.
-   * 
+   *
    * <pre>
    * // This rule is organized in a way that may not be most readable, but
    * // gives the best error messages.
    * classMemberDefinition
    *     : declaration ';'
    *     | methodDeclaration bodyOrNative
    *     ;
    *
    * // Note: this syntax is not official, but used in dart_interpreter. It
    * // is unlikely that Dart will support numbered natives.
@@ skipping 69 matching lines @@
    *     ;
    *
    * operator
    *     : unaryOperator
    *     | binaryOperator
    *     | '[' ']' { "[]".equals($text) }?
    *     | '[' ']' '=' { "[]=".equals($text) }?
    *     | NEGATE
    *     ;
    * </pre>
-   * 
+   *
    * @param allowStatic true if the static modifier is allowed
    * @return a {@link DartNode} representing the grammar fragment above
    */
   private DartNode parseFieldOrMethod(boolean allowStatic) {
     beginClassMember();
     Modifiers modifiers = Modifiers.NONE;
     if (optionalPseudoKeyword(STATIC_KEYWORD)) {
       if (!allowStatic) {
         reportError(position(), DartCompilerErrorCode.TOP_LEVEL_IS_STATIC);
       } else {
@@ skipping 2014 matching lines @@
         case RPAREN:
           expect(Token.RPAREN);
           --parenCount;
           break;
 
         case LPAREN:
           expect(Token.LPAREN);
           ++parenCount;
           break;
 
+        case EOS:
+          reportErrorWithoutAdvancing(DartCompilerErrorCode.UNEXPECTED_TOKEN);
+          return;
+
         case LBRACE:
         case SEMICOLON:
           return;
 
         default:
           next();
           break;
       }
     }
   }
@@ skipping 38 matching lines @@
       }
     }
   }
 
   /**
    * Collect plausible statement tokens and return a synthetic error statement
    * containing them.
    * <p>
    * Note that this is a crude heuristic that needs to be improved for better
    * error recovery.
-   * 
+   *
    * @return a {@link DartSyntheticErrorStatement}
    */
   private DartStatement parseErrorStatement() {
     StringBuilder buf = new StringBuilder();
     boolean done = false;
     int braceCount = 1;
     while (!done) {
       buf.append(getPeekTokenValue(0));
       next();
       switch (peek(0)) {
@@ skipping 34 matching lines @@
 
       case LBRACE:
         ++braceCount;
         //$FALL-THROUGH$
       default:
         // give error message
         expect(Token.SEMICOLON);
         break;
     }
 
-    while (true) { 
+    while (true) {
       switch (peek(0)) {
         case EOS:
           return;
 
         case SEMICOLON:
           if (braceCount < 2) {
             // if we have seen open braces while skipping, keep looking
             return;
           }
           next();
           break;
 
         case RBRACE:
           if (--braceCount == 0) {
             return;
           }
           break;
-     
+
         case LBRACE:
           ++braceCount;
           //$FALL-THROUGH$
         default:
           next();
           break;
       }
     }
   }
 
   /**
    * Report an error without advancing past the next token.
-   * 
+   *
    * @param errCode the error code to report, which may take a string parameter
    *     containing the actual token found
    */
   private void reportErrorWithoutAdvancing(DartCompilerErrorCode errCode) {
     startLookahead();
     Token actual = peek(0);
     next();
     reportError(position(), errCode, actual);
     rollback();
   }
@@ skipping 252 matching lines @@
 
       if (peek(0) == Token.CASE) {
         members.add(parseCaseMember(label));
       } else if (optional(Token.RBRACE)) {
         if (label != null) {
           reportError(position(), DartCompilerErrorCode.EXPECTED_CASE_OR_DEFAULT);
         }
         done = true;
         done(null);
       } else {
-        if (peek(0) != Token.EOS) { 
+        if (peek(0) != Token.EOS) {
           members.add(parseDefaultMember(label));
         }
         expectCloseBrace();
         done = true; // Ensure termination.
       }
     }
     return done(new DartSwitchStatement(expr, members));
   }
 
   /**
@@ skipping 270 matching lines @@
     } else {
       ctx.error(dartError);
       errorHistory.add(dartError.hashCode());
     }
   }
 
   private void reportError(DartNode node, ErrorCode errorCode, Object... arguments) {
     reportError(new DartCompilationError(node, errorCode, arguments));
   }
 }