Fix handling of 'c: if (0) break c; else ()' where a parser optimization

src/parser.cc

Index: src/parser.cc
===================================================================
--- src/parser.cc	(revision 10553)
+++ src/parser.cc	(working copy)
@@ -2128,6 +2128,10 @@
   // Parse labeled break statements that target themselves into
   // empty statements, e.g. 'l1: l2: l3: break l2;'
   if (!label.is_null() && ContainsLabel(labels, label)) {
+    // Consume the 'break' SEMICOLON if present
+    if (peek() == Token::SEMICOLON) {
+      scanner().Next();
+    }

[Lasse Reichstein Nielsen, 2012/02/02 10:05:19]

The semicolon *must* be there (or be inserted by automatic semicolon insertion),
so use 
  ExpectSemicolon(CHECK_OK);
like below.
Otherwise it'll still match something like
  foo: break foo var x = 10;

     return EmptyStatement();
   }
   BreakableStatement* target = NULL;