[ruby-dev:38161] Re: [feature:trunk] warning when Kernel#p is used

遠藤です。

2009/03/11 23:48 SASADA Koichi <[e-mail]>:
> 　ささだです．
>
> Yusuke ENDOH wrote::
>>   - 新しいオプションを追加する
>>   - 環境変数で指定する
>>   - RubyVM.warn_p = true という warn-p.rb を require する
>>
>> など、特にこだわりはないです。(最後のは名前以外わりといいかも？)
>>
>
> 　warn-p.rb を使いたい人が require する，だと足りないですかね．で，p を
> 再定義してしまえば．

ええと、提案のポイントは「p という名前のメソッドを関数呼び出しする
箇所を (静的に) 検出して警告する機能を入れよう」です。

alias my_p p として my_p を使うなどすると検出できなくなりますし、
自分で p というメソッドを定義して関数呼び出しすれば誤検出します。
が、(検出回避など以外の目的で) そんなことすることはなさそうですし、
この機能を使わなければ現状と何も変わりません。

で、この機能を有効にするトリガの案として、最初は「$DEBUG = true」を
挙げていましたが、これだと、検出は要らないけれど $DEBUG は使いたい
人が困るのでいまいちでした。
そこで改良案として、上では「warn-p を require する」をトリガとして
挙げています。


>>> アイデア自体は悪くないと思いますが、パーサレベルでチェックす
>>> るのであれば、syntax check (-c)などでひっかけるのがよいのでは
>>> ないかと思います。
>>
>> それでもいいと思います。
>
> 　パーサレベルでは難しいんじゃないですかね．

「p という名前のメソッドを関数呼び出しするかどうか」を検査するだけ
なのでできます。最初のメールにパッチがあります。

-- 
Yusuke ENDOH <[e-mail]>