Backport #4125: String#encode(:fallback) should accept default handler

• Status: Closed
• Assignee: yugui (Yuki Sonoda)

[ko1 (Koichi Sasada)]

=begin
　ささだです．

　最近，String#encode は :fallback オプションによって，置換できない文字
を置き換えるための文字（列）のテーブルを渡すことができる，ということを知
りました．

p "\u00A9".encode('sjis', fallback: {"\u00a9" => '(c)'})
#=> '(c)'

　ただ，fallback オプションは Hash のデフォルトハンドラを見てくれないよ
うです．

h = Hash.new{'?'}
h["\u00a9"] = '(c)'
p "\u00A9\u00AA".encode('sjis', fallback: h)
t.rb:3:in encode': U+00AA from UTF-8 to Shift_JIS (Encoding::UndefinedConversionError) from t.rb:3:in '

　マップを知っているものは特定の文字（列）に，知らないものはテキトーに
"?" とかにしたい，という用途はあるんじゃないかと思うので，対応して貰える
といいんじゃないかと思うんですが，どうでしょうか．

　Proc とか，#[] を持っているオブジェクト一般に広げてもいいかもしれませ
んが，その辺はお任せします．

--
// SASADA Koichi at atdot dot net

すでにパチモンがパッチを持ってるようですが...．

=end

[matz (Yukihiro Matsumoto)]

=begin
まつもと ゆきひろです

In message "Re: [ruby-dev:42692] [Feature: trunk] String#encode(:fallback) should accept default handler"
on Mon, 6 Dec 2010 21:04:18 +0900, SASADA Koichi [e-mail] writes:

|　ただ，fallback オプションは Hash のデフォルトハンドラを見てくれないよ
|うです．
|
|h = Hash.new{'?'}
|h["\u00a9"] = '(c)'
|p "\u00A9\u00AA".encode('sjis', fallback: h)
|t.rb:3:in encode': U+00AA from UTF-8 to Shift_JIS |(Encoding::UndefinedConversionError) | from t.rb:3:in '
|
|
|　マップを知っているものは特定の文字（列）に，知らないものはテキトーに
|"?" とかにしたい，という用途はあるんじゃないかと思うので，対応して貰える
|といいんじゃないかと思うんですが，どうでしょうか．
|
|
|　Proc とか，#[] を持っているオブジェクト一般に広げてもいいかもしれませ
|んが，その辺はお任せします．

|# すでにパチモンがパッチを持ってるようですが...．

この変更に賛成します。

=end

[naruse (Yui NARUSE)]

=begin
成瀬です。

2010年12月7日12:21 Yukihiro Matsumoto [e-mail]:

まつもと ゆきひろです

In message "Re: [ruby-dev:42692] [Feature: trunk] String#encode(:fallback) should accept default handler"
on Mon, 6 Dec 2010 21:04:18 +0900, SASADA Koichi [e-mail] writes:

|　ただ，fallback オプションは Hash のデフォルトハンドラを見てくれないよ
|うです．
|
|h = Hash.new{'?'}
|h["\u00a9"] = '(c)'
|p "\u00A9\u00AA".encode('sjis', fallback: h)
|t.rb:3:in encode': U+00AA from UTF-8 to Shift_JIS |(Encoding::UndefinedConversionError) | from t.rb:3:in '
|
|
|　マップを知っているものは特定の文字（列）に，知らないものはテキトーに
|"?" とかにしたい，という用途はあるんじゃないかと思うので，対応して貰える
|といいんじゃないかと思うんですが，どうでしょうか．
|
|
|　Proc とか，#[] を持っているオブジェクト一般に広げてもいいかもしれませ
|んが，その辺はお任せします．

|# すでにパチモンがパッチを持ってるようですが...．

この変更に賛成します。

ruby/ruby.h に追加している VALUE rb_obj_is_method(VALUE); に /intern/ をつけ、
テストを追加したら中田さんの持っていたパッチでいいんじゃないかと思います。
いれちゃってくださいませ

--
NARUSE, Yui
[e-mail]

=end

[ko1 (Koichi Sasada)]

=begin
(2010/12/07 16:43), NARUSE, Yui wrote:

ruby/ruby.h に追加している VALUE rb_obj_is_method(VALUE); に /intern/ をつけ、

　なら，別に ruby/ruby.h に載せる必要はないんじゃないでしょうか．

--
// SASADA Koichi at atdot dot net

=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

=begin
This issue was solved with changeset r30118.
Koichi, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

=end

[nobu (Nobuyoshi Nakada)]

• Tracker changed from Feature to Backport
• Project changed from Ruby master to Backport192
• Status changed from Closed to Assigned
• Assignee set to yugui (Yuki Sonoda)

[yugui (Yuki Sonoda)]

• Status changed from Assigned to Closed

I regard this issue as an enhancement. So I don't merge r30118.