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

Gábor Horváth xazax.hun at gmail.com
Thu Feb 19 02:00:07 PST 2015


Hi Aleksei,

According to Anna's comment, there are only two things left:

- Adding opt-in for inheritance and turning it off by default.
- Make it possible to query options for the package. There is one nontrivial design decision though. Should it be possible to query the options of another package (e.g.: should the Malloc checker be able to query the options of the osx package)?

Are you going to do these changes? If you need any help feel free to ping me. If you do not want to finish this patch I am volunteering to do it.


================
Comment at: lib/StaticAnalyzer/Core/AnalyzerOptions.cpp:114
@@ +113,3 @@
+      return StringRef(i->getValue());
+    size_t pos = Name.rfind('.');
+    if (pos == std::string::npos)
----------------
nit: Local variables should start with upper case letters.

http://reviews.llvm.org/D3967

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list