Forum: Ruby-dev [Feature: 1.9.2] rename rb_thread_* functions

Announcement (2017-05-07): www.ruby-forum.com is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see rubyonrails.org/community and ruby-lang.org/en/community for other Rails- und Ruby-related community platforms.
308cbef6e86dfc49cce3b2d4cf42aedc?d=identicon&s=25 SASADA Koichi (Guest)
on 2009-06-07 20:55
(Received via mailing list)
 ささだです.

 現在,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_... に変更してしまったんですが.



 パッチを添付します.ご検討下さい.
0ec4920185b657a03edf01fff96b4e9b?d=identicon&s=25 Yukihiro Matsumoto (Guest)
on 2009-06-08 00:04
(Received via mailing list)
$B$^$D$b$H(B $B$f$-$R$m$G$9(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 <ko1@atdot.net>
writes:

|$B!!8=:_!$(Bthread.c $B$GDj5A$5$l$F$$$k(B rb_thread_... 
$B$H$$$&4X?t$N$&$A!$$$$/$D(B
|$B$+$N4X?t$O(B rb_thread_t * $B$r0z?t$K$7$F$$$^$9!%$3$N$h$&$J4X?t$N(B prefix $B$r(B
|rb_th_... $B$K$7$F$bNI$$$G$7$g$&$+!%(B

$B$$$$$s$8$c$J$$$G$7$g$&$+!#%?%$%_%s%0$K$D$$$F$O(BYugui$B$5$s$NH=(B
$BCG$,I,MW$G$7$g$&$,!#(B
This topic is locked and can not be replied to.