From: Tomoyuki Chikanaga <nagachika00@...>
Date: 2011-12-05T10:14:40+09:00
Subject: [ruby-dev:44929] [Backport93 - Backport #5702] backport r33935


Issue #5702 has been updated by Tomoyuki Chikanaga.

File backport_5702.patch added

近永です。

そんなにコアじゃないですがレビューしました。
 * continue するよりは NameError 発生を else 節にしたほうが見通しが良いかと思います。好みの問題のような気もしますけど。
 * また NameError 発生時に、1つ以上の constant の flag を修正済みの場合に rb_clear_cache_by_class() が呼ばれていないことに気がつきました
というわけで追加の変更を添付します。


あと kosaki さんがおっしゃってたのは、差分の具体的な解説ではなくて(それがあったほうが良いのはもちろんでしょうけど)、「これは○○の件についてのbackport」と一言あると、日常的にチェックしてる人ならだいたいどの修正のことか見当がつくので自分が読めるかどうかすぐ判断ができていいということかなーって思います。この件なら「private constant のアレ」くらいで充分かと。違ってたらごめんなさい。

----------------------------------------
Backport #5702: backport r33935
http://redmine.ruby-lang.org/issues/5702

Author: Yusuke Endoh
Status: Open
Priority: Normal
Assignee: Yusuke Endoh
Category: 
Target version: 


r33886 を ruby_1_9_3 にバックポート希望です。
([ruby-list:48558] のバグ修正)
誰かレビュー頼みます。

-- 
Yusuke Endoh <mame@tsg.ne.jp>


-- 
http://redmine.ruby-lang.org