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

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 1 00:39:55 PDT 2022


JonasToth added a comment.

during the original implementation of the fix-util I had the plan to provide an option for east-vs-west const, but during the discussions we came to the conclusion that `clang-format` should do this (the patches were already pending at that point in time).

if this option works, i think its ok to provide it. having this check on during development is maybe just annoying when the `const` always pops up at the "wrong" side.

but i think we should provide more test cases here, especially for the edge cases where `const` _must_ be on the right side, because it would otherwise apply to something else then intended. that was the original reason why i sticked to east-const, because thats always correct.


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