[PATCH] [clang-tidy] Static Analyzer checker configuration options passthrough

Gábor Horváth xazax.hun at gmail.com
Wed Mar 11 05:06:33 PDT 2015


================
Comment at: clang-tidy/ClangTidy.cpp:218
@@ +217,3 @@
+        OptName.substr(AnalyzerPrefix.size(), Pos - AnalyzerPrefix.size());
+    AnalyzerOptions->Config[(Twine(AnalyzerCheckName) + ":" +
+                             OptName.substr(Pos + 1)).str()] = Opt.second;
----------------
alexfh wrote:
> Can we just use the same format in clang-tidy options (prepended with 'clang-analyzer-')? 
Unfortunately if colons are present in the key of a config option, there will be an yaml error message: unexpected ':'. It might be possible to escape it somehow but I find it an uglier solution, so I went ahead with the period syntax. It is more consistent with the tidy checker configuration syntax anyways.

http://reviews.llvm.org/D8164

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






More information about the cfe-commits mailing list