ruby-dev

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

Thread

Prev Next

In This Thread

Prev Next

[#44912] [ruby-trunk - Feature #4147] Array#sample で重みを指定したい — Shyouhei Urabe <shyouhei@...>

[#44918] [ruby-trunk - Bug #5699][Open] Windowsで一部のFloat値が正しく扱えない — 剛 奥山 <oku1111@...>

[#44925] [Backport93 - Backport #5702][Open] backport r33935 — Yusuke Endoh <mame@...>

[#44928] [ruby-trunk - Bug #5708][Open] Tempfile.new(pre) with block in test_autoclose — Akira Tanaka <akr@...>

[#44933] [ruby-trunk - Bug #5711][Assigned] r32774 (and r32783) broke nmake support — Usaku NAKAMURA <usa@...>

[#44940] Re: [ruby-cvs:41134] naruse:r33956 (trunk): Comment out tests which fails with GDBM-DBM compat mode. — Tanaka Akira <akr@...>

[#44950] [ruby-trunk - Bug #5725][Assigned] Coverage doesn't honor file path encodings — Shyouhei Urabe <shyouhei@...>

[#44951] [ruby-trunk - Bug #5728][Open] SystemExit.new(bool, message) — Akira Tanaka <akr@...>

[#44961] [ruby-trunk - Bug #5731][Open] enum_for を使うと method_missing にブロックが渡されない — Masahiro Tomita <tommy@...>

[#44962] [ruby-trunk - Feature #5735][Open] Extending the use of splat operator to when it is inside a hash — Tsuyoshi Sawada <sawadatsuyoshi@...>

[#44963] [ruby-trunk - Bug #5738][Open] if false; BEGIN {}; end が syntax error になる — Sho Hashimoto <sho-h@...>

[#44974] githubへの移行 — Shugo Maeda <shugo@...>

[#44983] [ruby-trunk - Bug #5748][Open] class_eval ブロック内での定数のスコープがドキュメントと矛盾 — Yasunori Miyamoto <m.yasunori.eip@...>

[#44984] [ruby-trunk - Bug #5753][Open] [Ripper] on_magic_comment doesn't pass proper value — Nobuyoshi Nakada <nobu@...>

[#44985] [ruby-trunk - Bug #5757][Open] main threadがreadやselectで待っていると、^C でなかなか死なない — Yui NARUSE <naruse@...>

[#44988] [ruby-trunk - Bug #5758][Open] webrickのhttpresponseについて — 寿宣 梶原 <kennyj@...>

[#44998] [Feature] st_update() — Nobuyoshi Nakada <nobu@...>

[#45000] [ruby-trunk - Bug #5762][Open] Bug#4178 が sparc Solaris 10 + Oracle Solaris Studio 12.2 で再発 — Naohisa Goto <ngotogenome@...>

[#45001] Rubyの次世代リファレンスマニュアルを構想する会 — Yutaka Hara <yutaka.hara@...>

[#45002] [ruby-trunk - Bug #5768][Assigned] TestRequire#test_race_exceptionで競合するケースがまだある — Yui NARUSE <naruse@...>

[#45008] Date.jd について — Yoshida Masato <yoshidam@...>

[#45016] [ruby-trunk - Bug #5783][Open] r33949からVC++6.0でminiprelude.c、newline.cの生成ができなくなった — Masahiro Kitajima <redmine@...>

[#45017] [ruby-trunk - Feature #5784][Open] Readline.special_prefixes の追加 — Tomoyuki Chikanaga <nagachika00@...>

[#45018] [ruby-trunk - Feature #5785][Open] Readline に pre_input_hook, insert_text, redisplay を追加 — Tomoyuki Chikanaga <nagachika00@...>

[#45021] [ruby-trunk - Bug #5786][Open] LoadError: cannot load such file -- openssl — Kazuhiro NISHIYAMA <redmine@...>

[#45030] [ruby-trunk - Bug #5790][Open] net/http の EOFError と Keep-Alive — Yui NARUSE <naruse@...>

[#45034] Re: [ruby-dev:44974] githubへの移行 — Daehyub Kim <lateau@...>

[#45040] [ruby-trunk - Bug #5801][Open] Enumerable#take_while の proc を外に出して使うと Segv — satoshi shiba <shiba@...>

[#45042] [ruby-trunk - Bug #5802][Open] 0 assretions in test/readline/test_readline.rb:test_line_buffer__point — Nobuhiro IMAI <nov@...>

[#45043] [ruby-trunk - Bug #5803][Open] closed output and Readline.readline dumps core — Nobuhiro IMAI <nov@...>

[#45048] [ruby-trunk - Bug #5808][Open] "a = []; a << a; puts JSON.dump(a)" aborted — Akira Tanaka <akr@...>

[#45051] [ruby-trunk - Bug #5812][Assigned] test-all is stuck on Windows — Usaku NAKAMURA <usa@...>

[#45057] [ruby-trunk - Feature #5820][Assigned] Merge Onigmo to Ruby 2.0 — Yui NARUSE <naruse@...>

[#45060] [ruby-trunk - Bug #5819][Assigned] Windowsで空のドライブのルートディレクトリに対するFile.stat が失敗する — Yui NARUSE <naruse@...>

[#45061] [ruby-trunk - Bug #5822][Open] testunit に -j オプションを渡すと稀にテストを実行せずに終了する — Tomoyuki Chikanaga <nagachika00@...>

[#45068] Re: [ruby-core:41846] [ruby-trunk - Feature #5361] Lice? — Ayumu Aizawa <ayumu.aizawa@...>

[ruby-dev:44929] [Backport93 - Backport #5702] backport r33935

Thread

In This Thread

[#44918] [ruby-trunk - Bug #5699][Open] Windowsで一部のFloat値が正しく扱えない — 剛奥山 <oku1111@...>

[#44988] [ruby-trunk - Bug #5758][Open] webrickのhttpresponseについて — 寿宣梶原 <kennyj@...>