[PATCH] D148461: [clang-tidy] Support C++17/20 in bugprone-exception-escape
Domján Dániel via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 7 02:04:22 PDT 2023
isuckatcs added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp:306
+
+ if (From->isMemberPointerType() || To->isMemberPointerType())
return false;
----------------
isuckatcs wrote:
> Please cover this line with both positive and negative test cases.
>
> Also upon looking up both [[ https://www.open-std.org/jtc1/sc22/wg21/docs/standards | N4849 ]] (C++ 20 draft) and [[ https://github.com/cplusplus/draft/releases | N4917]] (C++ 23 draft), they both say for qualification conversion that
>
>
> > each P_i is ... pointer to member of class C_i of type, ...
>
> Why are we not allowing them if the standard is at least C++ 20?
Please resolve this thread.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148461/new/
https://reviews.llvm.org/D148461
More information about the cfe-commits
mailing list