HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land.
Looks good, please wait for other reviewers. ================ Comment at: clang/unittests/Format/FormatTest.cpp:5747 +TEST_F(FormatTest, FormatAlignInsidePreprocessorElseBlock) { + FormatStyle Style = getLLVMStyleWithColumns(40); + Style.AlignConsecutiveAssignments.Enabled = true; ---------------- Why limit the columns? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134042/new/ https://reviews.llvm.org/D134042 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits