[PATCH] D16786: [Clang-tidy] Make modernize-redundant-void-arg working with included files

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 2 12:22:43 PST 2016


aaron.ballman added a comment.

In http://reviews.llvm.org/D16786#341938, @Eugene.Zelenko wrote:

> I didn't notice test cases for included files for other checkers. So it's hard to tell for should special test case introduced for this specific problem or not.


Generally speaking, all changes should have a test case accompanying them unless the change is NFC. It helps ensure we don't regress behavior the we want with the fix.


Repository:
  rL LLVM

http://reviews.llvm.org/D16786





More information about the cfe-commits mailing list