[PATCH] [Proposal] [Analyzer] Individual options for checkers

Anna Zaks zaks.anna at gmail.com
Fri Sep 26 17:59:27 PDT 2014


Aleksei,

Thanks for working on this! 

I've added a small comment above. 

The main problem with this patch is that it is lacking test cases. Is it possible to use it for the cases in CString checker and/or MallocChecker you've mentioned above?

Anna.

================
Comment at: include/clang/StaticAnalyzer/Core/AnalyzerOptions.h:274
@@ -249,3 +273,3 @@
   /// Interprets an option's string value as a boolean.
   ///
   /// Accepts the strings "true" and "false".
----------------
Please, add a comment about CheckerBase parameter in all the changed public methods.

http://reviews.llvm.org/D3967






More information about the cfe-commits mailing list