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