[PATCH] [clang-tidy] Default options in modules.

Alexander Kornienko alexfh at google.com
Thu Oct 16 00:26:43 PDT 2014


Ping
On 15 Oct 2014 16:14, "Alexander Kornienko" <alexfh at google.com> wrote:

> Hi djasper,
>
> This patch allows modules to specify default options for the checks
> defined in them. This way a sufficiently configurable check can be
> registered in
> multiple modules with different default options. E.g. the
> SpacesBeforeComments
> option may be set to 1 for the "llvm-namespace-comments" check and to 2
> for the
> "google-readability-namespace-comment" check without modifying or
> extending the
> check code.
>
> This patch also registers the google-readability-braces-around-statements
> check
> with suitable defaults.
>
> http://reviews.llvm.org/D5798
>
> Files:
>   clang-tidy/ClangTidyModule.cpp
>   clang-tidy/ClangTidyModule.h
>   clang-tidy/ClangTidyOptions.cpp
>   clang-tidy/ClangTidyOptions.h
>   clang-tidy/google/GoogleTidyModule.cpp
>   clang-tidy/readability/NamespaceCommentCheck.cpp
>   test/clang-tidy/google-module.cpp
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141016/74028786/attachment.html>


More information about the cfe-commits mailing list