[PATCH] [clang-tidy] Add a checker for swapped literal arguments.

Nico Weber thakis at chromium.org
Thu Jul 10 10:21:01 PDT 2014


Makes sense, thanks.


On Thu, Jul 10, 2014 at 9:40 AM, Daniel Jasper <djasper at google.com> wrote:

> I think for this (and probably many future clang-tidy checks), the answer
> "Does this make sense as a compiler warning?" is always: YES. However,
> clang-tidy enables us to quickly develop and tune an implementation using
> AST matchers, not worrying about slowing down the compiler and testing it
> on a larger codebase in order establish whether a check/warning provides
> sufficient benefit and even let users play around with it to get more
> feedback. So, I see clang-tidy as a prototyping platform and good checks
> will be turned into compiler warnings. At some stage we might want to think
> about how we can simplify that process, i.e. how we can make porting a
> clang-tidy check into a compiler warning easier, but IMO that stage isn't
> necessarily now.
>
> http://reviews.llvm.org/D4457
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140710/c53bbbd5/attachment.html>


More information about the cfe-commits mailing list