[PATCH] D59650: [NFC] ExceptionEscapeCheck: small refactoring

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 21 14:43:05 PDT 2019


JonasToth added inline comments.


================
Comment at: clang-tidy/utils/ExceptionAnalyzer.cpp:226
+ExceptionAnalyzer::ExceptionInfo
+ExceptionAnalyzer::analyzeBoilerplate(const T *Node) {
+  ExceptionInfo ExceptionList;
----------------
lebedev.ri wrote:
> Please bikeshed on the name. I don't think this one is good.
Hmm, `analyzeGeneric`, `analyzeGeneral`, `abstractAnalysis`, `analyzeAbstract`, something good in these?

Given its private its not too important either ;)


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D59650





More information about the cfe-commits mailing list