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

こういう変更を，理由の説明や事前の連絡無く行われると混乱します．
とりあえず revert して議論しませんか．

----

なお，私の立場を表明しておくと，私はこの変更に反対します．

推測するに，引数として任意のスレッドを撮るか，それとも current thread を
意味しているかわからない，ということかと思います（先日聞いた）．違ったら
下記の意見は見当違いなのでご指摘ください．

この観点でいうと，任意のスレッドを指定する名前を th から別の引数名に
し，th をカレントスレッドを渡す，という意味で使うべきだと主張します（一
部，すでに target_th という名前を使っています）．

理由は，他のソースコードにもこの名前規則（th == current thread）を使って
いるからです．あと，このケースを書くことが最も多いので，短い名前を利用し
たいです．


(2012/11/19 20:10), [e-mail] wrote:
> kosaki	2012-11-19 20:10:24 +0900 (Mon, 19 Nov 2012)
> 
>   New Revision: 37730
> 
>   http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=37730
> 
>   Log:
>     * thread.c, vm_core.h: big rename th to cur_th when works only
>       th is current thread.
> 
>   Modified files:
>     trunk/ChangeLog
>     trunk/thread.c
>     trunk/vm_core.h
> 


-- 
// SASADA Koichi at atdot dot net