Feature #3995: Hash#update with Enumerable

• Status: Rejected
• Assignee: -
• Target version: -

[nobu (Nobuyoshi Nakada)]

=begin
[ruby-dev:42476]でMLからの登録に失敗したので再登録します

今のところHash#updateはHashしか受け付けませんが、Enumerableまで
受け付けるように拡張するのはどうでしょうか。
=end

[matz (Yukihiro Matsumoto)]

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

In message "Re: [ruby-dev:42477] [Ruby 1.9-Feature#3995][Open] Hash#update with Enumerable"
on Thu, 28 Oct 2010 19:17:23 +0900, Nobuyoshi Nakada [e-mail] writes:

|今のところHash#updateはHashしか受け付けませんが、Enumerableまで
|受け付けるように拡張するのはどうでしょうか。

絶対反対とまではいいませんが、eachが返すべきものがなにかとか
やや明確さに欠けるような気がします。

=end

[nobu (Nobuyoshi Nakada)]

=begin
なかだです。

At Thu, 28 Oct 2010 22:51:37 +0900,
Yukihiro Matsumoto wrote in [ruby-dev:42478]:

|今のところHash#updateはHashしか受け付けませんが、Enumerableまで
|受け付けるように拡張するのはどうでしょうか。

絶対反対とまではいいませんが、eachが返すべきものがなにかとか
やや明確さに欠けるような気がします。

eachがyieldするべきものはHash[]と合わせたつもりです。

使い道としては、重複のチェックとか。

appearance = open(file) {|f|
{}.update(f.each_line.with_index) {|s, o, n| [*o, n]}
}

--
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
中田 伸悦

=end

[matz (Yukihiro Matsumoto)]

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

In message "Re: [ruby-dev:42483] Re: [Ruby 1.9-Feature#3995][Open] Hash#update with Enumerable"
on Fri, 29 Oct 2010 04:02:57 +0900, Nobuyoshi Nakada [e-mail] writes:

|> 絶対反対とまではいいませんが、eachが返すべきものがなにかとか
|> やや明確さに欠けるような気がします。
|
|eachがyieldするべきものはHash[]と合わせたつもりです。

RB_TYPE_P()を使ってはいけないような気がします。せめて、
rb_check_array_type() を使うべきではないでしょうか。

|使い道としては、重複のチェックとか。
|
| appearance = open(file) {|f|
| {}.update(f.each_line.with_index) {|s, o, n| [*o, n]}
| }

うーん、上記の実装の問題がなかったとしても、なんかユースケー
スとして弱い気がするのですが。

=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Rejected
• Description updated (diff)