[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