Project

General

Profile

Feature #4783

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

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

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

Description

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

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

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

Associated revisions

Revision 31752
Added by Kouhei Sutou over 4 years ago

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

Revision 31752
Added by Kouhei Sutou over 4 years ago

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

History

#1 [ruby-dev:43583] Updated by Kouhei Sutou over 4 years ago

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

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

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

#2 [ruby-dev:43585] Updated by Shyouhei Urabe over 4 years ago

  • Status changed from Assigned to Closed

pullしておきました。

#3 [ruby-dev:43588] Updated by Kouhei Sutou over 4 years ago

ありがとうございます!

#4 [ruby-dev:43589] Updated by Kouhei Sutou over 4 years ago

  • % Done changed from 0 to 100

Also available in: Atom PDF