Project

General

Profile

Backport #7989

ISeq#to_a line info bug

Added by ko1 (Koichi Sasada) almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
[ruby-core:53039]

Description

Please backport r39536:

* iseq.c (iseq_data_to_ary): fix condition.
  r34303 introduces a bug to avoid all line information from
  a result of ISeq#to_a. This is a regression problem from 2.0.0p0.
* test/ruby/test_iseq.rb: add a test of lines after ISeq#to_a.

Related r39539 is a new bug fix. But not mature.

Associated revisions

Revision a5e89ea4
Added by nagachika (Tomoyuki Chikanaga) over 6 years ago

merge revision(s) 39536: [Backport #7989]

    * iseq.c (iseq_data_to_ary): fix condition.
      r34303 introduces a bug to avoid all line information from
      a result of ISeq#to_a. This is a regression problem from 2.0.0p0.

    * test/ruby/test_iseq.rb: add a test of lines after ISeq#to_a.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@39910 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 39910
Added by nagachika (Tomoyuki Chikanaga) over 6 years ago

merge revision(s) 39536: [Backport #7989]

* iseq.c (iseq_data_to_ary): fix condition.
  r34303 introduces a bug to avoid all line information from
  a result of ISeq#to_a. This is a regression problem from 2.0.0p0.

* test/ruby/test_iseq.rb: add a test of lines after ISeq#to_a.

History

Updated by nagachika (Tomoyuki Chikanaga) almost 7 years ago

  • Status changed from Open to Assigned
  • Assignee set to nagachika (Tomoyuki Chikanaga)
#2

Updated by nagachika (Tomoyuki Chikanaga) over 6 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r39910.
Koichi, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


merge revision(s) 39536: [Backport #7989]

* iseq.c (iseq_data_to_ary): fix condition.
  r34303 introduces a bug to avoid all line information from
  a result of ISeq#to_a. This is a regression problem from 2.0.0p0.

* test/ruby/test_iseq.rb: add a test of lines after ISeq#to_a.

Also available in: Atom PDF