Re: [ruby-gnome2-devel-ja] enumやflagsの規則について

[Masaaki A. <mas...@gm...>]

青柳です。

>  * require 'vte/deprecated.rb'ではなく、
>    require 'vte/deprecated'にしてもらえますか？

修正しました。


>  * GNOMEUtilsというよりは、GLib::XXXの中に入れたほうがいい
>    かなぁと思いました。

いい場所と名前は、ありますか？
今のところ思いつかなくて。。。


>  * define_XXXは外に出してよいと思いますが、constant_getは外
>    に出したくないかなぁと思いました。

privateに変更しました。


>  * 定数のときもwarningを出したほうが便利かも。
>    const_missingを使って、最初にアクセスされたときにwarning
>    出しながらconst_setするとそんな動作になりそう。

const_missingを使って、warning出力するようにしました。
ただ、const_defined?で確認して何かやっているようなコードがあると問題が出ますが、
ないですよね。


>> また、match_set_cursor～ についてIFをまとめた方が良いと思います。
>> 添付のパッチのようにまとめて、既にリリース済みのmatch_set_cursor_typeについては
>> deprecatedとしてwarningを出すようにするのは、いかがでしょうか？
>
> その通りだと思います。
> warnのメッセージの中にcallerの結果も入れた方が便利かなぁと思
> いました。メッセージがでてもどこで出ているかわからないと直す
> のに時間がかかってしまうので。

warningが出てるのを分かりつつ使うということもあると思うので、デフォルトでcallerまで含めるのは
止めておいた方がいいように思います。
何かしらの設定で、callerも出力できるようにするというのは、いいかもしれないですが


以上の内容でコミットしましたので、ご確認ください。