commit 73a9849f38fbc3e66435bd14ac1cd50dcae7d6b6
author Laurent Le Brun <laurentlb@google.com> Tue Mar 17 15:46:19 2015 +0000
committer Kristina Chodorow <kchodorow@google.com> Wed Mar 18 13:42:44 2015 +0000
tree c2c94f946d17229d8691481f31741b8fba828372
parent 361af116d5d3966b2f94c7435eaa36dc2e7b4713

Skylark: Fix an infinite loop in the parser

--
MOS_MIGRATED_REVID=88830705

src/main/java/com/google/devtools/build/lib/syntax/Parser.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
index 555f08a..56a32e9 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
@@ -958,7 +958,7 @@
     return list;
   }
 
-  // load(STRING (COMMA STRING)*)
+  // load '(' STRING (COMMA STRING)* COMMA? ')'
   private void parseLoad(List<Statement> list) {
     int start = token.left;
     if (token.kind != TokenKind.STRING) {
@@ -974,7 +974,7 @@
       symbols.add(new Ident((String) token.value));
     }
     expect(TokenKind.STRING);
-    while (token.kind != TokenKind.RPAREN) {
+    while (token.kind != TokenKind.RPAREN && token.kind != TokenKind.EOF) {
       expect(TokenKind.COMMA);
       if (token.kind == TokenKind.RPAREN) {
         break;