[PATCH] D23279: clang-reorder-fields
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 9 09:07:52 PDT 2016
aaron.ballman added a comment.
In https://reviews.llvm.org/D23279#510002, @omtcyfz wrote:
> In https://reviews.llvm.org/D23279#509896, @compnerd wrote:
>
> > If the argument really is that we want to minimize the tools then Id argue that `clang-rename` also belongs in `clang-tidy` as it would be used to rename fields to match the naming convention (tidying up your code base).
>
>
> It does not belong to `clang-tidy`. `clang-tidy` is a linter, it is meant for diagnosing and fixing typical programming errors.
We have the modernize and readability modules which don't really deal with programming errors at all (for some definition of programming error).
Repository:
rL LLVM
https://reviews.llvm.org/D23279
More information about the cfe-commits
mailing list