[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:33:57 PDT 2020


Typz added a comment.

In D85600#2209545 <https://reviews.llvm.org/D85600#2209545>, @Typz wrote:

> Could this also be causing https://bugs.llvm.org/show_bug.cgi?id=33896 ?
>
> I'll try to see if changes something in that case...

Actually, I can confirm this is unrelated.



================
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.
----------------
bad indent here : should be space-indented.


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