[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