[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