lebedev.ri added a comment.
To me, modulo the `"IgnoredExceptions"` change, this seems like a
straight-forward refactoring.
================
Comment at: clang-tidy/utils/ExceptionAnalyzer.h:25
+/// give the possibility of an exception.
+class ExceptionAnalyzer {
+public:
----------------
What is the test plan for this utility class, BTW?
Ensuring that the main user (`bugprone-exception-escape`) has sufficiently full
test coverage?
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57100/new/
https://reviews.llvm.org/D57100
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
