[PATCH] D85600: [clang-format] use spaces for alignment of binary/ternary expressions with UT_AlignWithSpaces

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 11 04:51:44 PDT 2020


Typz added inline comments.


================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:1360
+      if (Style.AlignOperands == FormatStyle::OAS_AlignAfterOperator)
+        NewParenState.UnindentOperator = true;
+      // Mark indentation as alignment if the expression is aligned.
----------------
fickert wrote:
> Typz wrote:
> > bad indent here : should be space-indented.
> Can you clarify? This is formatted with the LLVM clang-format.
Phabricator shows some kind of `>>` symbol at the beginning of this line, which i interpreted as a TAB character, hence my comment.
But indeed no issue after looking at the raw file, sorry about this.
  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85600



More information about the cfe-commits mailing list