[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