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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
