Project

General

Profile

Actions
Copy link

Bug #11338

closed

[PATCH] iseq.c (iseq_data_to_ary): dump kw_arg as symbol

Added by normalperson (Eric Wong) almost 9 years ago. Updated over 8 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
Backport:
2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: DONE
[ruby-core:69891]

Description

iseq.c (iseq_data_to_ary): dump kw_arg as symbol

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

Files

0001-iseq.c-iseq_data_to_ary-dump-kw_arg-as-symbol.patch (1.64 KB) 0001-iseq.c-iseq_data_to_ary-dump-kw_arg-as-symbol.patch normalperson (Eric Wong), 07/07/2015 07:41 PM
Actions
Copy link
 #1

Updated by Anonymous almost 9 years ago

  • Status changed from Open to Closed

Applied in changeset r51190.

iseq.c (iseq_data_to_ary): dump kw_arg as symbol

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]
Actions
Copy link
 #2 [ruby-core:71729]

Updated by nagachika (Tomoyuki Chikanaga) over 8 years ago

  • Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: REQUIRED to 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: DONE

Backported into ruby_2_2 branch at r52783.

Actions
Copy link

Also available in: Atom PDF

Like0
Like0Like0