[PATCH] D32478: [clang-format] Fix AlignOperands when BreakBeforeBinaryOperators is set

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 29 06:07:05 PDT 2017


Typz added inline comments.


================
Comment at: include/clang/Format/Format.h:167
+    /// \endcode
+    OAS_StrictAlign,
+  };
----------------
djasper wrote:
> The name is not intuitive. I don't think this is any more or less strict than the other version.
It is a bit stricter in the sense that it really aligns operands, not operator with first operand... But this is indeed not very intuitive.

Any suggestion?


https://reviews.llvm.org/D32478





More information about the cfe-commits mailing list