commit 5f6744550a98506ed680b389cfdc872fe07daed5
author Laurent Le Brun <laurentlb@google.com> Tue Mar 17 19:29:13 2015 +0000
committer Kristina Chodorow <kchodorow@google.com> Wed Mar 18 13:45:21 2015 +0000
tree 4310f649a03a24da973e1f0090b7cf392589d731
parent 6683d645ff3a1f3aedc4b7d559b9fbe4d7bcafc4

Parser: allow multiple statements on the same line as the colon

 e.g. def foo(x, y): print(x); print(y)

Bug found when adding the pass statement.

--
MOS_MIGRATED_REVID=88852710

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 56a32e9..6a856f6 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
@@ -1210,6 +1210,7 @@
     return ImmutableList.copyOf(arguments);
   }
 
+  // suite is typically what follows a colon (e.g. after def or for).
   // suite ::= simple_stmt
   //         | NEWLINE INDENT stmt+ OUTDENT
   private List<Statement> parseSuite() {
@@ -1227,9 +1228,7 @@
       }
       expect(TokenKind.OUTDENT);
     } else {
-      Statement stmt = parseSmallStatement();
-      list.add(stmt);
-      expect(TokenKind.NEWLINE);
+      parseSimpleStatement(list);
     }
     return list;
   }