Issue #7651 has been updated by okkez (okkez _).

Status changed from Open to Assigned
Assignee set to okkez (okkez _)

そのまま取り込みたいところなんですが、複数の変更が混ざってるのでこちらで分割して取り込みます。
ありがとうございます！
----------------------------------------
Bug #7651: bc-checkparams.rb の改良

https://bugs.ruby-lang.org/issues/7651#change-35693

Author: pypypy567 (py _)
Status: Assigned
Priority: Normal
Assignee: okkez (okkez _)
Category: 
Target version: 
reporter: 
ruby_version: 


http://bugs.ruby-lang.org/projects/rurema/wiki/Phase3ReviewProcess1
wiki のこのページには

> @param が全部書いてあるか? -> bc-checkparams で確認できる
とあります。
しかし実際コードを読んでみると @param にある名前がメソッドシグネチャに含まれてるか見てるだけで逆はチェックしてないようにみえます。
これだと @param の直後の引数名の typo くらいしか検出できません。
他にもいくつか細かい問題が有ります。

* @todo に対応してなくて [UNKNOWN_META_INFO] と言われる
* 複数行のメソッドシグネチャに対応してない（多分最後の行以外無視される）
* [Feature #6989] の形式に対応してない

以上について適当に対応してみました。
method_info の定義内の rescue についてはどうするか考えるのが面倒になって手付かずです。

一応手元ではそれなりに動いてる気がします。
パッチの適用前後で _builtin 以下の全ファイルについて機械的に適用してみた結果も添付します。

# 他にもプリプロセス後の行番号を表示するせいでソースの位置がわかりにくいとかもありますがそこまでは手が出ませんでした。



-- 
http://bugs.ruby-lang.org/

--
ML: ruby-reference-manual@m...
Info: http://QuickML.com/