Project

General

Profile

Bug #13120

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

Added by yahonda (Yasuo Honda) over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
ruby -v:
ruby 2.5.0dev (2017-01-10 trunk 57299) [x86_64-linux]
[ruby-core:79039]

Description

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

Related issues

Related to Ruby master - Bug #11977: Bug with array literals caused by r53376Closednormalperson (Eric Wong)Actions
Is duplicate of Ruby master - Bug #13114: test failures since r57284Closednobu (Nobuyoshi Nakada)Actions
#1

Updated by shyouhei (Shyouhei Urabe) over 3 years ago

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

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • Status changed from Open to Closed

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • Status changed from Closed to Open
#4

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • Status changed from Open to Closed

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

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

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

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

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • 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]

Updated by yahonda (Yasuo Honda) over 3 years ago

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.

Updated by naruse (Yui NARUSE) about 3 years ago

  • 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.

Also available in: Atom PDF