A new cop for checking eval's arguments have fname and lineno

[pocke]

eval 時の FILE, LINE + 1 を強制する cop ってあったっけ？ @onk https://twitter.com/onk/status/935350994375868416

Eval族は引数にファイル名、行番号を与えることでスタックトレースのファイル名などをいい感じにすることができます。 https://docs.ruby-lang.org/ja/latest/method/Kernel/m/eval.html https://docs.ruby-lang.org/ja/latest/method/Module/i/module_eval.html これを行っていないとスタックトレースがわかりづらくなってしまうため、ファイル名などを渡すように強制するCopがほしいです。

実装は簡単そうですが、いくつか注意点があります。

• eval と class_eval では引数が違う
  • evalはbindingを受け取るため
• 行番号などを違う形で指定する必要があるケースもある
  • 例: https://github.com/bbatsov/rubocop/blob/master/lib/rubocop/node_pattern.rb#L533
  • ひとつ目の引数が文字列リテラルでなければ、このCopを適用しないようにすれば良いかも知れない。
  • ひとつ目の引数が文字列リテラルでなければ、どこにコードが書いてあるかとかわからないので
• 渡されるソースコードがheredoc なら __LINE__ + 1, 普通の文字列リテラルなら__LINE__を渡す必要がありそう

[pocke]

https://github.com/bbatsov/rubocop/pull/5173 pull-requested

[pocke]

merged