Bug #13120: p [].class shows ThreadSafe::Array when it expects to show Array

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: ruby 2.5.0dev (2017-01-10 trunk 57299) [x86_64-linux]
• Backport: 2.2: DONTNEED, 2.3: DONTNEED, 2.4: DONE

[yahonda (Yasuo Honda)]

It has been filed as Rails issue https://github.com/rails/rails/issues/27631 and found there was a similar bug reported https://bugs.ruby-lang.org/issues/11977 . It looks it reproduces again.

Steps to reproduce:

$ ruby -v
ruby 2.5.0dev (2017-01-10 trunk 57299) [x86_64-linux]
$ bundle -v
Bundler version 1.13.7
$ git clone -b ruby250 https://github.com/yahonda/array_bug.git
$ cd array_bug
$ bundle
$ bundle exec ruby -w -e 'p [].class'
Array
$ bundle exec ruby -w -e 'require "rails"; p [].class'
ThreadSafe::Array

[shyouhei (Shyouhei Urabe)]

• Related to Bug #11977: Bug with array literals caused by r53376 added

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

[nobu (Nobuyoshi Nakada)]

• Status changed from Closed to Open

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

[nobu (Nobuyoshi Nakada)]

• Status changed from Closed to Open
• Backport changed from 2.2: UNKNOWN, 2.3: UNKNOWN, 2.4: UNKNOWN to 2.2: DONTNEED, 2.3: DONTNEED, 2.4: REQUIRED

[nobu (Nobuyoshi Nakada)]

• Is duplicate of Bug #13114: test failures since r57284 added

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

Applied in changeset r57305.

---

variable.c: fix the condition to cache

• variable.c (rb_const_set): fix the condition to cache the class
  path and cache permanent or temporary path corresponding to the
  outer klass. [ruby-core:79039] [Bug #13120]

[yahonda (Yasuo Honda)]

Verified that it has been addressed by using revision 57306.

$ ruby -v
ruby 2.5.0dev (2017-01-11 trunk 57306) [x86_64-linux]
$ bundle -v
Bundler version 1.13.7
$ git clone -b ruby250 https://github.com/yahonda/array_bug.git
$ cd array_bug
$ bundle
$ bundle exec ruby -w -e 'p [].class'
Array
$ bundle exec ruby -w -e 'require "rails"; p [].class'
Array
$

Thanks.

[naruse (Yui NARUSE)]

• Backport changed from 2.2: DONTNEED, 2.3: DONTNEED, 2.4: REQUIRED to 2.2: DONTNEED, 2.3: DONTNEED, 2.4: DONE

ruby_2_4 r57856 merged revision(s) 57305.