aaron.ballman added a comment.
Can you add test coverage that demonstrates the fix behaves as expected?
================
Comment at: clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp:454-455
+ StringRef FileName(Loc.printToString(Loc.getManager()));
+ if(getHeaderFilter()->match(FileName))
+ {
+ Converter.emitDiagnostic(Loc, DiagLevel, Message, Info.getRanges(),
----------------
Formatting is incorrect here -- you should run the patch through clang-format
(https://clang.llvm.org/docs/ClangFormat.html#script-for-patch-reformatting),
and elide the braces.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59135/new/
https://reviews.llvm.org/D59135
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
