njames93 added inline comments.

================
Comment at: 
clang-tools-extra/clang-tidy/bugprone/UnhandledExceptionAtNewCheck.cpp:23-24
+    const CXXCatchStmt *CatchS = Node.getHandler(I);
+    if (InnerMatcher.matches(CatchS->getCaughtType(), Finder, Builder))
+      return true;
+    // Generic catch handler should match anything.
----------------
This can leak bound nodes If it doesn't match.


================
Comment at: 
clang-tools-extra/clang-tidy/bugprone/UnhandledExceptionAtNewCheck.h:28
+  bool isLanguageVersionSupported(const LangOptions &LangOpts) const override {
+    return LangOpts.CPlusPlus;
+  }
----------------
I'd suggest this check is only ran when exceptions are enabled.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97196/new/

https://reviews.llvm.org/D97196

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to