lebedev.ri marked an inline comment as done.
lebedev.ri added inline comments.


================
Comment at: clang/lib/Format/Format.cpp:768
   LLVMStyle.CommentPragmas = "^ IWYU pragma:";
+  LLVMStyle.ConstStyle = FormatStyle::CS_Leave;
   LLVMStyle.CompactNamespaces = false;
----------------
MyDeveloperDay wrote:
> 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)
There is already a number of stylistic LLVM-specific decisions in LLVM style
* 80-col wrap
* pointer alignment to the right
* spaces instead of tabs
* ???
I think it first and foremost is an LLVM style, and i don't
really recall any guarantees that it must not/will not change.


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

https://reviews.llvm.org/D69764



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to