ignore 'break' statements

bschmalhofer · Mar 25, 2009 · dc2e166 · dc2e166
1 parent fa40af7
commit dc2e166
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/src/pct/actions.pm b/src/pct/actions.pm
@@ -859,6 +859,10 @@ method param_list($/) {
     make $block;
 }
 
+method break($/) {
+    make PAST::Stmts.new( :name('break') );
+}
+
 method empty_statement($/) {
     make PAST::Stmts.new( :name('empty statement') );
 }

diff --git a/src/pct/grammar.pg b/src/pct/grammar.pg
@@ -124,6 +124,7 @@ token close_short_tag {
 # statements
 
 rule statement {
+    | <break>                          {*}  #= break
     | <namespace_definition>           {*}  #= namespace_definition
     | <return_statement>               {*}  #= return_statement
     | <require_once_statement>         {*}  #= require_once_statement
@@ -163,6 +164,9 @@ rule empty_statement {
     ';' {*}
 }
 
+rule break {
+    'break' <.statement_delimiter> {*}
+}
 rule namespace_definition {
     'namespace' <namespace_name>?  {*}       #= open
     <block>                        {*}       #= close
@@ -215,8 +219,8 @@ rule else_clause {
 rule switch_statement {
     'switch' '(' <expression> ')'
     '{'
-        [ 'case' <literal> ':' <statement_list>  ]*
-        [ 'default'        ':' <default=statement_list>  ]?
+        [ 'case' <literal> ':' <statement_list> ]*
+        [ 'default'        ':' <default=statement_list> ]?
     '}' {*}
 }
 

diff --git a/t/php/control_flow.t b/t/php/control_flow.t
@@ -323,6 +323,7 @@ language_output_is( 'Pipp', <<'CODE', '', 'switch, single non-matching case' );
 switch (22) {
     case 11:
        echo 'no match';
+       break;
 }
 
 ?>
@@ -334,6 +335,7 @@ language_output_is( 'Pipp', <<'CODE', 'match', 'switch, single matching case' );
 switch (22) {
     case 22:
        echo 'match';
+       break;
 }
 
 ?>
@@ -345,6 +347,7 @@ language_output_is( 'Pipp', <<'CODE', 'default', 'switch, default case' );
 switch (22) {
     case 22:
        echo 'default';
+       break;
 }
 
 ?>
@@ -356,8 +359,10 @@ language_output_is( 'Pipp', <<'CODE', 'twentytwo', 'switch, two cases' );
 switch (22) {
     case 21:
        echo 'twentyone';
+       break;
     case 22:
        echo 'twentytwo';
+       break;
 }
 
 ?>