diff --git a/ChangeLog b/ChangeLog index dc4345a2ef..1f726bd624 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Tue Jan 8 14:41:41 2013 Nobuyoshi Nakada + + * vm_core.h (rb_iseq_t): move flip_cnt from struct iseq_compile_data, + because it has same life span as enclosing iseq. [Bug #7671] + [ruby-core:51296] + Mon Jan 7 23:43:00 2013 Kenta Murata * ext/bigdecimal/bigdecimal.c (rmpd_power_by_big_decimal): diff --git a/compile.c b/compile.c index 7dffe08141..35feaa3122 100644 --- a/compile.c +++ b/compile.c @@ -5051,12 +5051,11 @@ iseq_compile_each(rb_iseq_t *iseq, LINK_ANCHOR *ret, NODE * node, int poped) LABEL *lend = NEW_LABEL(nd_line(node)); LABEL *lfin = NEW_LABEL(nd_line(node)); LABEL *ltrue = NEW_LABEL(nd_line(node)); - struct iseq_compile_data *data = iseq->local_iseq->compile_data; + rb_iseq_t *local_iseq = iseq->local_iseq; rb_num_t cnt; VALUE key; - if (!data) data = iseq->compile_data; - cnt = data->flip_cnt++ + DEFAULT_SPECIAL_VAR_COUNT; + cnt = local_iseq->flip_cnt++ + DEFAULT_SPECIAL_VAR_COUNT; key = INT2FIX(cnt); ADD_INSN2(ret, nd_line(node), getspecial, key, INT2FIX(0)); diff --git a/iseq.h b/iseq.h index 4a7965adf9..0790529638 100644 --- a/iseq.h +++ b/iseq.h @@ -98,7 +98,6 @@ struct iseq_compile_data { struct iseq_compile_data_storage *storage_current; int last_line; int last_coverable_line; - int flip_cnt; int label_no; int node_level; const rb_compile_option_t *option; diff --git a/test/ruby/test_flip.rb b/test/ruby/test_flip.rb index 8d7cdd03ad..bd14228a8c 100644 --- a/test/ruby/test_flip.rb +++ b/test/ruby/test_flip.rb @@ -10,4 +10,12 @@ class TestFlip < Test::Unit::TestCase 2000.times {eval %[(foo..bar) ? 1 : 2]} end end + + def test_shared_eval + bug7671 = '[ruby-core:51296]' + vs = (1..9).to_a + vs.select {|n| if n==2..n==16 then 1 end} + v = eval("vs.select {|n| if n==3..n==6 then 1 end}") + assert_equal([*3..6], v, bug7671) + end end diff --git a/vm_core.h b/vm_core.h index 6ecbd70289..f1dc013313 100644 --- a/vm_core.h +++ b/vm_core.h @@ -306,6 +306,7 @@ struct rb_iseq_struct { /* misc */ ID defined_method_id; /* for define_method */ + rb_num_t flip_cnt; /* used at compile time */ struct iseq_compile_data *compile_data;