[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

carlosgalvezp added a comment. There's a lot of changes in this patch (at least 6 as per commit message) - please split into smaller patches fixing 1 problem at a time. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145865/new/ https://reviews.llvm

[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

PiotrZSL updated this revision to Diff 513691. PiotrZSL added a comment. Ping + Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145865/new/ https://reviews.llvm.org/D145865 Files: clang-tools-extra/clang-tidy/bugprone/ExceptionEscapeCheck.c

[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

PiotrZSL added a comment. This change obsoletes D115118 and D134588 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145865/new/ https://reviews.llvm.org/D145865

[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

PiotrZSL updated this revision to Diff 504448. PiotrZSL added a comment. Update documentation Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145865/new/ https://reviews.llvm.org/D145865 Files: clang-tools-extra/clang-tidy/bugprone/ExceptionEscape

[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

PiotrZSL updated this revision to Diff 50. PiotrZSL added a comment. Review fixes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145865/new/ https://reviews.llvm.org/D145865 Files: clang-tools-extra/clang-tidy/bugprone/ExceptionEscapeCheck.cp

[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp:491 if (const auto *ThrownExpr = Throw->getSubExpr()) { - const auto *ThrownType = - ThrownExpr->getType()->getUnqualifiedDesugaredType(); - if (Thro

[PATCH] D145865: [clang-tidy] Multiple fixes to bugprone-exception-escape

PiotrZSL created this revision. Herald added subscribers: carlosgalvezp, xazax.hun. Herald added a reviewer: njames93. Herald added a project: All. PiotrZSL requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. - Added support for C++