archive/ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2025-09-15 08:33:58 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

iseq.c (iseq_data_to_ary): dump kw_arg as symbol

Browse source 
Fix RubyVM::InstructionSequence#to_a after r49517
Keywords were made symbols to fix [Bug #10831] [ruby-core:68031],
so we should dump symbols as-is instead of attempting to convert
them from IDs

* iseq.c (iseq_data_to_ary): dump kw_arg as symbol
* test/-ext-/iseq_load/test_iseq_load.rb: test kw_arg roundtrip
  [ruby-core:69891] [Bug #11338]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51190 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
normal 2015-07-07 19:50:49 +00:00
parent fe94eaa5ae
commit 011347e4d7
3 changed files with 16 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

9
test/-ext-/iseq_load/test_iseq_load.rb
View file

@ -86,6 +86,15 @@ class TestIseqLoad < Test::Unit::TestCase
assert_iseq_roundtrip(src)
end
def test_kwarg
assert_iseq_roundtrip <<-'end;'
def foo(kwarg: :foo)
kwarg
end
foo(kwarg: :bar)
end;
end
# FIXME: still failing
def test_require_integration
skip "iseq loader require integration tests still failing"