[PATCH] D13398: [clang-tidy] add check cppcoreguidelines-pro-type-const-cast

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 9 11:24:59 PDT 2015


On 7 Oct 2015 19:24, "Aaron Ballman" <aaron.ballman at gmail.com> wrote:
>
> On Tue, Oct 6, 2015 at 11:07 PM, Alexander Kornienko <alexfh at google.com>
wrote:
> > alexfh added a comment.
> >
> > It looks like potentially we're going to have tens of checks in this
module. I wonder whether we should start organizing the checks somehow
right away. For example, create a directory (and a namespace) for each
profile. We'd need to adapt the add_new_check.py script to support this
then.
>
> I wouldn't be opposed to that. Would this organization carry over to
> other things as well, like the documentation and test directories?

I don't feel strongly about the layout of the tests directory.

For the documentation I would like to keep a flat directory structure so
that a URL of a documentation page for a check could be constructed by
simply substituting the check name to a simple template. This way
clang-tidy frontends can easily point users to the relevant documentation
without adding meta-information specifically for this purpose.

>
> ~Aaron
>
> >
> > Just a thought. The patch LG.
> >
> >
> > http://reviews.llvm.org/D13398
> >
> >
> >
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151009/a137175a/attachment.html>


More information about the cfe-commits mailing list