[PATCH] D69764: [clang-format] Add East Const / West Const fixer

Mon Nov 4 00:05:18 PST 2019

MyDeveloperDay marked 2 inline comments as done.
MyDeveloperDay added inline comments.

================
Comment at: clang/lib/Format/Format.cpp:768
   LLVMStyle.CommentPragmas = "^ IWYU pragma:";
+  LLVMStyle.ConstStyle = FormatStyle::CS_Leave;
   LLVMStyle.CompactNamespaces = false;
----------------
lebedev.ri wrote:
> Based on code reviews, this should be `CS_West`.
I would tend to agree for the LLVM project itself, but I really don't feel I can change this due to the huge number of projects which have BasedOnStyle: LLVM  (not to mention most other style inherit from LLVM like Google, Chromium etc...)

I think this needs to be an opt-in change in the .clang-format (at least for now)

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

https://reviews.llvm.org/D69764