Project

General

Profile

Feature #1422

Unused function rb_ary_to_s

Added by ujihisa (Tatsuhiro Ujihisa) over 10 years ago. Updated over 8 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Target version:
[ruby-dev:38359]

Description

=begin
ujihisaといいます。

Array#to_sがArray#inspectのaliasになったため、rb_ary_to_sがruby処理系内のどこからも呼ばれていません。
http://redmine.ruby-lang.org/repositories/revision/1?rev=10879

RubyレベルでArray#to_sとArray#inspectを両方用意しておくのはよいとして、拡張ライブラリのためrb_ary_inspectとrb_ary_to_sの両方を用意しておく必要はないのではないかと思うのですが、どうでしょうか。対称性のためCレベルでも全てのクラスにto_sとinspectを用意しておく必要があるならば、このメールは無視してください。また、この変更で既存のライブラリがどの程度動かなくなるかも分かりません。

rb_ary_to_sを削除するパッチを添付します。
=end


Files

0001-Remove-unused-function-rb_ary_to_s.patch (1.04 KB) 0001-Remove-unused-function-rb_ary_to_s.patch ujihisa (Tatsuhiro Ujihisa), 04/30/2009 10:28 AM

History

#1

Updated by matz (Yukihiro Matsumoto) over 10 years ago

  • Status changed from Open to Rejected

=begin

=end

#2

Updated by matz (Yukihiro Matsumoto) over 10 years ago

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

In message "Re: [ruby-dev:38359] [Feature #1422] Unused function rb_ary_to_s"
on Thu, 30 Apr 2009 10:29:03 +0900, T UJIHISA redmine@ruby-lang.org writes:

|Array#to_sがArray#inspectのaliasになったため、rb_ary_to_sがruby処理系内のどこからも呼ばれていません。
|http://redmine.ruby-lang.org/repositories/revision/1?rev=10879
|
|RubyレベルでArray#to_sとArray#inspectを両方用意しておくのはよいとして、拡張ライブラリのためrb_ary_inspectとrb_ary_to_sの両方を用意しておく必要はないのではないかと思うのですが、どうでしょうか。対称性のためCレベルでも全てのクラスにto_sとinspectを用意しておく必要があるならば、このメールは無視してください。また、この変更で既存のライブラリがどの程度動かなくなるかも分かりません。

必要性はないとは思いますが、拡張ライブラリがrb_ary_to_sを呼
んでいる可能性があるため残してあるような気がしました。当面残
しておこうと思います。

=end

Also available in: Atom PDF