[PATCH] D97196: [clang-tidy] Add new check 'bugprone-unhandled-exception-at-new'.
Balázs Kéri via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 13 02:43:03 PDT 2021
balazske added inline comments.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone-unhandled-exception-at-new.rst:7
+Finds calls to ``new`` that may throw ``std::bad_alloc`` exception and
+the exception handler is missing.
+
----------------
aaron.ballman wrote:
> This isn't quite accurate -- if the exception handler is missing or doesn't handle either `std::bad_alloc` or `std::exception`.
I restructure the text, but do not want to mention `std::exception` because it is a base class (handling of `std::exception` means handling of `std::bad_alloc`), and then why not mention generic exception handler (`catch(...)`). I think this text is for introduction only, should not be totally precise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97196/new/
https://reviews.llvm.org/D97196
More information about the cfe-commits
mailing list