[Feature: 1.9.2] rename rb_thread_* functions


#1

 ささだです.

 現在,thread.c で定義されている rb_thread_… という関数のうち,いくつ
かの関数は rb_thread_t * を引数にしています.このような関数の prefix を
rb_th_… にしても良いでしょうか.

メリット:
・rb_thread_… を見たら,Thread インスタンスを渡す関数だとわかる.
・rb_th_… を見たら,VM インターナルな関数だとわかる.
(そういう意味では,rb_vm_… でもいいかもしれないが,
 rb_vm_t * が引数な関数と見分けが付かない)

懸念:

・static 関数への影響

 あまり問題ないと考えられます.

・!static 関数への影響

 バイナリ互換性の問題が生じますが,そもそも rb_thread_t * を得る関数が
ないので,拡張ライブラリから rb_thread_t * を引数とする関数は呼べなかっ
たわけです(ごにょごにょすると取れますが,それもちょっと).というわけ
で,問題ないんじゃないかなぁ,と思います.

 ただ,include/ruby/intern.h に,

void rb_thread_signal_raise(void *, int);
void rb_thread_signal_exit(void *);

なんてのがあったんですが,これは外から使うんでしょうか.とりあえず,
rb_th_… に変更してしまったんですが.

 パッチを添付します.ご検討下さい.


#2

e$B$^$D$b$He(B e$B$f$-$R$m$G$9e(B

In message “Re: [ruby-dev:38575] [Feature: 1.9.2] rename rb_thread_*
functions”
on Mon, 8 Jun 2009 03:54:28 +0900, SASADA Koichi removed_email_address@domain.invalid
writes:

|e$B!!8=:!$e(Bthread.c e$B$GDj5A$5$l$F$$$ke(B rb_thread… e$B$H$$$&4X?t$N$&$A!$$$$/$De(B
|e$B$+$N4X?t$Oe(B rb_thread_t * e$B$r0z?t$K$7$F$$$^$9!%$3$N$h$&$J4X?t$Ne(B prefix e$B$re(B
|rb_th_… e$B$K$7$F$bNI$$$G$7$g$&$+!%e(B

e$B$$$$$s$8$c$J$$$G$7$g$&$+!#%?%$%_%s%0$K$D$$$F$Oe(BYuguie$B$5$s$NH=e(B
e$BCG$,I,MW$G$7$g$&$,!#e(B