[ruby-dev:45937] Re: [ruby-cvs:43524] kosaki:r36348 (trunk): * include/ruby/ruby.h: Removed RUBY_GLOBAL_SETUP complely. It is

2012/7/9 Nobuyoshi Nakada <[e-mail]>:
> なかだです。
>
> At Tue, 10 Jul 2012 05:39:10 +0900,
> SASADA Koichi wrote in [ruby-dev:45928]:
>>　 こちら，残しておいてもいいかもしれない，とちょっと思っています．
>>
>> (1) 何かしら，今後定義したいかもしれない
>> (2) 空の定義を残しておけば，これまでのアプリ組込みのコードが動く
>>
>>　 (1) のような，「今後あるかも」系の提案は evil かもしれません
>> が...．core だけの話なら，後で revert すればいいんですが，libruby を使っ
>> てる人のコードがあることを考えると躊躇する次第です．

なにか誤解があると思うのですが、このコミット前ですと空マクロなので例え、
librubyをリンクしているアプリケーションがが RUBY_GLOBAL_SETUP を使っていたとしてもNOPに
展開されてしまっているので、今後librubyの RUBY_GLOBAL_SETUP が
置き換わったとしても動作はしません。libruby系の話をするなら ruby_platform_setup()
みないな関数を作って、かつ組み込みアプリケーションはそれを必ず呼ばないといけないというようなドキュメントがないとうまくいかないと考えています。意味のある動作をしているコードがあるならコードを読んで理解しろという態度も成り立つ余地もありますが現状NOPなわけですし。

> 削除には反対します。ここで互換性を壊す必要があるほどジャマなコー
> ドでもないし、実際[Feature #5053]のチェックを実装してみる際にも
> 使ったりしています。
>
> 元のMacOSの分岐は無意味になっているので消して構わないと思います
> が、@deprecatedというコメントはrevertしなくていいです。

つぎにこちらについてですが、どうにも使い道がない、それほど邪魔ではないのどちらともTrueであると考えているのでrevert自体には反対しません。しかしこれを他の目的に再利用することには反対します。マクロがライブラリ境界を超えている事自体が良くないので、こいつはそのまま誰にも使われてないまま死んでいくべき。

[Feature #5053]
のチケットにはなかださんのパッチは添付されていないようですので、そちらにはコメントできませんが普通に関数呼び出しで比較するようなつくりに出来ない理由が思いつきません。


追伸： オブトピックですが、現状5053ではRUBY_DESCRIPTION
を比較する案が提案されてますが、コンパイルオプションを変えて性能比較する最中にミスるとかのケースを考えると単にコンパイル時刻を埋め込んで比較するのがシンプルでいいんじゃないかな