[PATCH] D119599: Add option to align compound assignments like `+=`

[sstwcw via Phabricator via cfe-commits]

sstwcw marked 5 inline comments as done.
sstwcw added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:16872
+  // that things still get aligned.
+  Alignment.ColumnLimit = 20;
   EXPECT_EQ("int    a(int   x,\n"
----------------
sstwcw wrote:
> curdeius wrote:
> > Is it something that can be done/fixed separately?
> I added a revision here https://reviews.llvm.org/D119625.  If you commit it separately there will be a merge conflict though.
I hadn't noticed that column limit = 0 meant no limit. This test doesn't really need to change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119599