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

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

Reply via email to