[ruby-dev:43124] Re: Hashのイテレーション中の新規キー追加

成瀬です。

まず、わたしは「Hashのイテレーション中の新規キー追加」には条件付き賛成です。
どんな条件かは後述しますが、たぶんまめさんやうささんと同じでしょう。

さて、[ruby-dev:43111] のパッチですが、いくつか重大な誤りがあります。
一つは近永さんの指摘と同じですが、
* 追加に際して rehash が発生した場合、イテレーションが壊れる
  (つまり、1.8 で追加を禁止している理由にぶつかっている)
* 追加した要素をその each 中に実行するべきか考える
* 1.8 では each の評価順が実装依存

もっとも、これは簡単な回避策がありまして、以下のような実装にすると、まとめて解決できます。
class Hash
  def each
    keys = hash.keys
    keys.each do |key|
      yield(key, hash[key]) if hash.key?(key)
    end
  end
end

もう一つはより重大で、このパッチが完成しても 1.8.7 には入らない、ということです。
また、1.8.8 も [ruby-dev:43064] あたりで出さない方向で合意が形成されつつあります。
http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-dev/43064
つまり、1.8 系に新機能を追加するという方向は現状あまりないでしょう。

また、1.9.2 向けというのもやっぱりなくて（メンテナンスブランチになるのでこれも新機能追加はない）、
trunk (=Ruby 1.9) 向けが正解です。
http://redmine.ruby-lang.org/wiki/ruby/HowToContribute

ここで戻って 1.9 向けのパッチ後の挙動がどうあるべきかを考えると、現在の 1.9 の仕様として、
* 1.9 の Hash#each は挿入順で評価される（仕様）
があるので、あるべき挙動は、
* each 中のハッシュへの要素追加を常に許す
* 追加した要素は最後に評価する
となるべきだ、というのは割と合意の取れる線かなぁと思います。
で、このようなパッチが実際に作られたならば、わたしはそのマージに賛成します。
（これ以外の仕様だったら反対するという意味ではない）

-- 
NARUSE, Yui  <[e-mail]>