diff options
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | bootstraptest/test_syntax.rb | 10 | ||||
-rw-r--r-- | compile.c | 5 |
3 files changed, 20 insertions, 2 deletions
@@ -1,3 +1,10 @@ +Wed May 19 19:45:10 2010 Yusuke Endoh <mame@tsg.ne.jp> + + * iseq_compile_each (NODE_DEFINED): put nil first to fix stack + consistency. [ruby-core:30293] + Now, lfinish[0] of defined_expr seems not to be used. Refactoring + may be needed. + Wed May 19 16:55:09 2010 NARUSE, Yui <naruse@ruby-lang.org> * numeric.c (rb_num2ulong): use rb_big2ulong for data from diff --git a/bootstraptest/test_syntax.rb b/bootstraptest/test_syntax.rb index 44db11babf..ae1488353e 100644 --- a/bootstraptest/test_syntax.rb +++ b/bootstraptest/test_syntax.rb @@ -890,3 +890,13 @@ assert_valid_syntax('x y { "#{}".z { } }', bug1240) assert_valid_syntax('x y { "#{}".z do end }', bug1240) assert_valid_syntax('y "#{a 1}" do end', '[ruby-core:29579]') +assert_normal_exit %q{ + def foo(&block) + yield + end + + foo do + s = defined?(raise + 1) + Class + end +}, '[ruby-core:30293]' @@ -4815,11 +4815,12 @@ iseq_compile_each(rb_iseq_t *iseq, LINK_ANCHOR *ret, NODE * node, int poped) LABEL *lfinish[2]; lfinish[0] = NEW_LABEL(nd_line(node)); lfinish[1] = 0; + ADD_INSN(ret, nd_line(node), putnil); defined_expr(iseq, ret, node->nd_head, lfinish, Qtrue); + ADD_INSN(ret, nd_line(node), swap); + ADD_INSN(ret, nd_line(node), pop); if (lfinish[1]) { - ADD_INSNL(ret, nd_line(node), jump, lfinish[0]); ADD_LABEL(ret, lfinish[1]); - ADD_INSN(ret, nd_line(node), putnil); } ADD_LABEL(ret, lfinish[0]); } |