owenpan accepted this revision. owenpan added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang/lib/Format/WhitespaceManager.cpp:407 + // We should not remove required spaces unless we break the line before. + assert(Changes[i].NewlinesBefore || + Changes[i].Spaces >= ---------------- ================ Comment at: clang/unittests/Format/FormatTest.cpp:17287 + " return 2;\n" + "}};\n", + BracedAlign); ---------------- ================ Comment at: clang/unittests/Format/FormatTest.cpp:17298 + " return 2;\n" + "} };\n", + BracedAlign); ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119419/new/ https://reviews.llvm.org/D119419 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits