Feature #4783

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

Added by Shyouhei Urabe almost 4 years ago. Updated almost 4 years ago.

[ruby-dev:<unknown>]
Status:Closed
Priority:Normal
Assignee:Shyouhei Urabe

Description

lib/rexmlのメンテナの須藤さん:

pull requestが来ています。パッチ自体は妥当かなという気がしますので、そもそも現状が意図された状態なのか、それともこれはバグってことにするのか、その辺ご判断いただければと。よろしくおねがいします。

https://github.com/ruby/ruby/pull/22

Associated revisions

Revision 31752
Added by Kouhei Sutou almost 4 years ago

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

Revision 31752
Added by Kouhei Sutou almost 4 years ago

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

History

#1 Updated by Kouhei Sutou almost 4 years ago

  • Assignee changed from Kouhei Sutou to Shyouhei Urabe
  • Target version set to 2.0.0

引数の文字列オブジェクトに興味があるのではなくて、文字列の中身だけに興味があると思うので、freezeは落としちゃう方向でよいと思います。なので、取り込みたいと思うのですが、pullってもらってよいでしょうか?

テストは直したほうがよいと思うので、取り込んでもらった後に修正します。

#2 Updated by Shyouhei Urabe almost 4 years ago

  • Status changed from Assigned to Closed

pullしておきました。

#3 Updated by Kouhei Sutou almost 4 years ago

ありがとうございます!

#4 Updated by Kouhei Sutou almost 4 years ago

  • % Done changed from 0 to 100

Also available in: Atom PDF