Starting from http://reviews.llvm.org/rL231266 options for individual static analyzer checkers can be specified.
This patch makes it possible to specify static analyzer checker options in clang tidy configuration files.
Differential D8164
[clang-tidy] Static Analyzer checker configuration options passthrough xazax.hun on Mar 9 2015, 4:50 AM. Authored by
Details Starting from http://reviews.llvm.org/rL231266 options for individual static analyzer checkers can be specified. This patch makes it possible to specify static analyzer checker options in clang tidy configuration files.
Diff Detail Event TimelineComment Actions One minor question: the options are stored in std::map<std::string, std::string>. I wonder if there is any specific reason not to use llvm::StringMap in this case. Comment Actions I don't remember any specific reason. You can try to replace map<> with StringMap in a separate patch.
Comment Actions Changes:
|
I'd say that the function _sets_ static analyzer checker options, not gets them. Also, IMHO the other order of arguments would be better, as it seems more intuitive if the source comes first (it may be a matter of taste, though, and memcpy, for example, puts destination first).