From b52f1af0d8d9267cfb4abd5cacd41d0f75105e4e Mon Sep 17 00:00:00 2001 From: shugo Date: Fri, 9 Oct 2015 03:58:34 +0000 Subject: * compile.c (iseq_compile_each): Dynamic string literals (e.g., "#{x}") should not be frozen because they don't literally represent strings. https://twitter.com/shugomaeda/status/651937650027401216 https://twitter.com/yukihiro_matz/status/651942882312482817 https://twitter.com/yukihiro_matz/status/651980835181096960 git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@52087 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 9 +++++++++ compile.c | 5 ----- test/ruby/test_iseq.rb | 13 +++++++------ 3 files changed, 16 insertions(+), 11 deletions(-) diff --git a/ChangeLog b/ChangeLog index 3a9c2fb207..9e5897360b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +Fri Oct 9 12:52:08 2015 Shugo Maeda + + * compile.c (iseq_compile_each): Dynamic string literals (e.g., + "#{x}") should not be frozen because they don't literally + represent strings. + https://twitter.com/shugomaeda/status/651937650027401216 + https://twitter.com/yukihiro_matz/status/651942882312482817 + https://twitter.com/yukihiro_matz/status/651980835181096960 + Fri Oct 9 06:52:49 2015 KOSAKI Motohiro * benchmark/prepare_require.rb: skip file creation if it already diff --git a/compile.c b/compile.c index 69abb591ce..c1ebb8fc00 100644 --- a/compile.c +++ b/compile.c @@ -5059,11 +5059,6 @@ iseq_compile_each(rb_iseq_t *iseq, LINK_ANCHOR *ret, NODE * node, int poped) if (poped) { ADD_INSN(ret, line, pop); } - else { - if (iseq->compile_data->option->frozen_string_literal) { - ADD_SEND (ret, line, idFreeze, INT2FIX(0)); - } - } break; } case NODE_XSTR:{ diff --git a/test/ruby/test_iseq.rb b/test/ruby/test_iseq.rb index 72b9c4ae3a..e6028da488 100644 --- a/test/ruby/test_iseq.rb +++ b/test/ruby/test_iseq.rb @@ -166,12 +166,13 @@ class TestISeq < Test::Unit::TestCase $f = 'f' line = __LINE__ + 2 code = <<-'EOS' - ['foo', 'foo', "#{$f}foo"] + ['foo', 'foo', "#{$f}foo", "#{'foo'}"] EOS - s1, s2, s3 = RubyVM::InstructionSequence.compile(code, __FILE__, __FILE__, line, {frozen_string_literal: true}).eval - assert(s1.frozen?) - assert(s2.frozen?) - assert(s3.frozen?) - assert(s1.object_id == s2.object_id) + s1, s2, s3, s4 = RubyVM::InstructionSequence.compile(code, __FILE__, __FILE__, line, {frozen_string_literal: true}).eval + assert_equal(true, s1.frozen?) + assert_equal(true, s2.frozen?) + assert_equal(false, s3.frozen?) + assert_equal(true, s4.frozen?) + assert_equal(s2.object_id, s2.object_id) end end -- cgit v1.2.3