NoQ added inline comments.

================
Comment at: clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.h:192-197
+  /// Pair of checker name and enable/disable to do analysis.
+  std::vector<std::pair<std::string, bool>> CheckerAnalysisVector;
+
+  /// Vector of checker names to do not emit warnings.
+  std::vector<std::string> CheckerSilenceVector;
 
----------------
`CheckerAnalysisVector` sounds like a vector of checkers that will be subject 
to analysis. But in reality they are the ones that are analyzing and nobody is 
analyzing them.

The old name isn't very expressive either, but at least it sounds cool >.<

Maybe `EnabledCheckers`, `SilencedCheckers`?


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

https://reviews.llvm.org/D66042



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

Reply via email to