commit 59f587a2e756e28767e31dacf2c0131a27683aa3
author Laurent Le Brun <laurentlb@google.com> Mon Mar 16 14:51:36 2015 +0000
committer Han-Wen Nienhuys <hanwen@google.com> Mon Mar 16 17:35:13 2015 +0000
tree 7c8014a05e61d46035c6347269ee79a1dedf8bad
parent ce59704dc567e32918118377bacabf031e6117a5

Skylark: Allow traling comma in load statements

--
MOS_MIGRATED_REVID=88726457

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 90f79ec..45eae4c 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
@@ -977,8 +977,11 @@
       symbols.add(new Ident((String) token.value));
     }
     expect(TokenKind.STRING);
-    while (token.kind == TokenKind.COMMA) {
+    while (token.kind != TokenKind.RPAREN) {
       expect(TokenKind.COMMA);
+      if (token.kind == TokenKind.RPAREN) {
+        break;
+      }
       if (token.kind == TokenKind.STRING) {
         symbols.add(new Ident((String) token.value));
       }