Feature #4783

REXML::Text.new 引数がfreezeされていたとき

Added by shyouhei (Shyouhei Urabe) 12 months ago. Updated 12 months ago.

[ruby-dev:<unknown>]
Status:Closed Start date:05/26/2011
Priority:Normal Due date:
Assignee:shyouhei (Shyouhei Urabe) % Done:

100%
Category:lib
Target version:2.0.0

Description

lib/rexmlのメンテナの須藤さん: pull requestが来ています。パッチ自体は妥当かなという気がしますので、そもそも現状が意図された状態なのか、それともこれはバグってことにするのか、その辺ご判断いただければと。よろしくおねがいします。 https://github.com/ruby/ruby/pull/22

Associated revisions

Revision 31752
Added by kou (Kouhei Sutou) 12 months ago

* test/rexml/test_core.rb (Tester::test_text_frozen): split frozen string test. refs #4783

Revision 31752
Added by kou (Kouhei Sutou) 12 months ago

* test/rexml/test_core.rb (Tester::test_text_frozen): split frozen string test. refs #4783

History

[ruby-dev:43583]#1
Updated by kou (Kouhei Sutou) 12 months ago

  • Assignee changed from kou (Kouhei Sutou) to shyouhei (Shyouhei Urabe)
  • Target version set to 2.0.0
引数の文字列オブジェクトに興味があるのではなくて、文字列の中身だけに興味があると思うので、freezeは落としちゃう方向でよいと思います。なので、取り込みたいと思うのですが、pullってもらってよいでしょうか？ テストは直したほうがよいと思うので、取り込んでもらった後に修正します。

[ruby-dev:43585]#2
Updated by shyouhei (Shyouhei Urabe) 12 months ago

  • Status changed from Assigned to Closed
pullしておきました。

[ruby-dev:43588]#3
Updated by kou (Kouhei Sutou) 12 months ago

ありがとうございます！

[ruby-dev:43589]#4
Updated by kou (Kouhei Sutou) 12 months ago

  • % Done changed from 0 to 100

Also available in: Atom PDF