[ruby-dev:46529] Re: Fwd: [ruby-changes:25559] shugo:r37616 (trunk): * vm_core.h (rb_call_info_t::refinements), compile.c (new_callinfo):

前田です。

2012年11月16日 17:52 SASADA Koichi <[e-mail]>:
>> refineのブロック内でメソッド定義するたびにメソッドエントリを追加又は変更するものと思って
>> いましたが、prependベースの案だとそうしなくてよいんです?
>> # prependベースだと追加だけでよくなると思いますが。
>
>
> 　少なくとも今回の例は，2回目の refine まで X::foo は追加されないと思う
> ので何も問題無いと思うんですが，勘違いでしょうか．

prependベースにしなくても問題ないということですよね。
私もそう思うのですが、なかださんは違う実装のイメージなのかなあと思って↑のように書きました。

>> どちらかというと、rb_method_definition_t::bodyに元のメソッドエントリを保存する方法だと
>> メモリ管理の面で面倒(メソッドエントリはメモリ管理が特殊なので)な気がしてきたので、
>> そういう意味でprependベースの方がすっきりするかもしれないですね。
>
> 　えーと，特殊かなぁ．たんに，refine type な definition が消えたときに，
> 一緒に original_definition を消せばいいだけのような．

rb_method_entry_tじゃなくてrb_method_definition_tの情報だけ保存して、flagとかは
refinedなメソッドエントリ(rb_method_entry_t)の方に持たせるということですか?
オリジナルのrb_method_entry_tを保存しておくという話だと思っていたので面倒そうだな
と思っていました。

original_definitionを消すというのはalias_countが0になったら、ということで合ってます?
呼び出し時には新しいrb_method_entry_tを用意してrefinedなメソッドエントリの情報を
コピーする感じですかね。

> 　いや，やってみないとわからないかもしれないですが，何か複雑怪奇なリファ
> レンスを持ち回るような場合はわからんですが，そうなる例が思いつきませんで
> した．
>
> 　そういえば alias ってどうなんだろう．

refinedなメソッドのaliasを作る時は、refinedじゃない方のrb_method_definiton_tを引っぱって
来てaliasのエントリを作るのかなと思っていましたが、そういう話ではなく?