[PATCH] D131386: [clang-tidy] Added `ConstAlignment` option to `misc-const-correctness`

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 15 07:46:39 PDT 2022


aaron.ballman added a comment.

In D131386#3723144 <https://reviews.llvm.org/D131386#3723144>, @carlosgalvezp wrote:

>> but removing those options can break people's .clang-tidy config files
>
> Aren't there deprecation mechanisms? I think trying to be backwards compatible across all possible versions can lead to suboptimal solutions and make the tool harder to use and hard/slow to adapt to the needs of the community.

Nothing stops us from deprecating config options with some nice diagnostic behavior, but again, this pushes the work off onto the user to maintain their scripts and gives them an inconsistent interface to clang-tidy where some checks get formatting options and other checks do not.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131386/new/

https://reviews.llvm.org/D131386



More information about the cfe-commits mailing list