[ruby-dev:46550] Re: [ruby-cvs:44900] kosaki:r37730 (trunk): * thread.c, vm_core.h: big rename th to cur_th when works only

2012/11/19 SASADA Koichi <[e-mail]>:
> (2012/11/19 22:15), SASADA Koichi wrote:
>>>>> >> > 理由は，他のソースコードにもこの名前規則（th == current thread）を使って
>>>>> >> > いるからです．あと，このケースを書くことが最も多いので，短い名前を利用し
>>>>> >> > たいです．
>>> > 短くしたいというなら curかなあ。thはありえないと思います。説明が必要な
>>> > 変数の命名規則というのがどうも。変数名って説明するためにあるんでしょ。
>>> > という感覚
>>　 私としては，現状での全体の整合性を取りたいので，th で統一したいと思い
>> ます．
>
> 　改めて見直してみたんですが，th を使っていても current thread のものが
> まだ thread.c に混ざっているようです．実は，見ていて cur_th にしているの
> もわかりやすいなぁ，とも思ってしまったのですが，任意の thread を受け取
> る，というものを target_th というような別の名前にしたほうが，作業量を少
> なく全体を統一出来るのではないかとも感じました（すでに，他のところではそ
> の混乱があります）．
>
> 　現状が，th という引数名で2つの意味がありわかりづらい，というのは同意し
> ますので，target_th というような名前にするのが良いのではないかと思います．
>
> 　rb_thread_t *th = GET_THREAD()
>
> というイディオムが至る所にあるので，この辺がそもそも thread って何だよ，
> とか突っ込みが入りそうなところです．とりあえず，現状だとまだ混乱している
> ので，一度リセットして，ある程度まとめてから作業に取りかかるのがいいん
> じゃないかと思います．それが 2.0 に入れるべき作業であるかどうかはわかり
> ません．
>
> 　混乱をなくすという観点から，thread.c に関しては，複数のスレッドが入り
> 乱れるので，th という変数名を禁止する，という方針がいいのかもしれませ
> ん．とりあえず，現状の th を任意のスレッドを受け取る，という意味で使って
> いるのは受け入れがたいです．

ちょっと議論が今日中に収束する気がしないので、いったんrevertしますね。