diff --git a/parse.y b/parse.y index b76f3ca00d..ef9f68ae34 100644 --- a/parse.y +++ b/parse.y @@ -152,7 +152,7 @@ struct local_vars { struct vtable *past; # endif struct local_vars *prev; - stack_type cmdargs; + stack_type cmdargs, cond; /* XXX: backup for cmdargs_stack and cond_stack. Because this is not a part of local variables, refactoring is needed. */ }; #define DVARS_INHERIT ((void*)1) @@ -2900,31 +2900,22 @@ primary : literal $$ = in_def; in_def = 1; } - { - $1 = cond_stack; - $$ = cmdarg_stack; - COND_SET(0); - CMDARG_SET(0); - } f_arglist bodystmt k_end { /*%%%*/ - NODE *body = remove_begin($7); + NODE *body = remove_begin($6); reduce_nodes(&body); - $$ = NEW_DEFN($2, $6, body, &@$); - nd_set_line($$->nd_defn, @8.end_pos.lineno); + $$ = NEW_DEFN($2, $5, body, &@$); + nd_set_line($$->nd_defn, @7.end_pos.lineno); set_line_body(body, @1.beg_pos.lineno); /*% - $$ = dispatch3(def, $2, $6, $7); + $$ = dispatch3(def, $2, $5, $6); %*/ local_pop(); in_def = $4 & 1; current_arg = $3; - - COND_SET($1); - CMDARG_SET($5); } | k_def singleton dot_or_colon {SET_LEX_STATE(EXPR_FNAME);} fname { @@ -2935,31 +2926,22 @@ primary : literal $$ = current_arg; current_arg = 0; } - { - $1 = cond_stack; - $$ = cmdarg_stack; - COND_SET(0); - CMDARG_SET(0); - } f_arglist bodystmt k_end { /*%%%*/ - NODE *body = remove_begin($9); + NODE *body = remove_begin($8); reduce_nodes(&body); - $$ = NEW_DEFS($2, $5, $8, body, &@$); - nd_set_line($$->nd_defn, @10.end_pos.lineno); + $$ = NEW_DEFS($2, $5, $7, body, &@$); + nd_set_line($$->nd_defn, @9.end_pos.lineno); set_line_body(body, @1.beg_pos.lineno); /*% - $$ = dispatch5(defs, $2, $3, $5, $8, $9); + $$ = dispatch5(defs, $2, $3, $5, $7, $8); %*/ local_pop(); in_def = $4 & 1; current_arg = $6; - - COND_SET($1); - CMDARG_SET($7); } | keyword_break { @@ -10675,6 +10657,8 @@ local_push_gen(struct parser_params *parser, int inherit_dvars) # endif local->cmdargs = cmdarg_stack; CMDARG_SET(0); + local->cond = cond_stack; + COND_SET(0); lvtbl = local; } @@ -10696,6 +10680,7 @@ local_pop_gen(struct parser_params *parser) vtable_free(lvtbl->args); vtable_free(lvtbl->vars); CMDARG_SET(lvtbl->cmdargs); + COND_SET(lvtbl->cond); xfree(lvtbl); lvtbl = local; } diff --git a/test/ruby/test_syntax.rb b/test/ruby/test_syntax.rb index bf9f4998ac..a1af048a2b 100644 --- a/test/ruby/test_syntax.rb +++ b/test/ruby/test_syntax.rb @@ -1186,6 +1186,11 @@ eom assert_valid_syntax('while def foo a = tap do end; end; break; end') end + def test_classdef_in_cond + assert_valid_syntax('while class Foo; tap do end; end; break; end') + assert_valid_syntax('while class Foo a = tap do end; end; break; end') + end + private def not_label(x) @result = x; @not_label ||= nil end