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

Mark de Wever via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 14 08:56:51 PDT 2022


Mordante added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp:10
 #include "ConstCorrectnessCheck.h"
 #include "../utils/FixItHintUtils.h"
 #include "clang/AST/ASTContext.h"
----------------
This is already included in the header.


================
Comment at: clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp:66
+      ConstAlignment(
+          Options.get("ConstAlignment", utils::fixit::QualifierPolicy::Right)) {
   if (AnalyzeValues == false && AnalyzeReferences == false)
----------------
I would suggest to use `QualifierAlignment` to match the name in clang-format.


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