MyDeveloperDay added a comment. My personal preference is a new struct without Enabled
================ Comment at: clang/lib/Format/Format.cpp:1196 LLVMStyle.AlignConsecutiveMacros = {}; + LLVMStyle.AlignConsecutiveTrailingComments = {}; + LLVMStyle.AlignConsecutiveTrailingComments.Enabled = true; ---------------- Won’t you need to initialise the members here as above ================ Comment at: clang/unittests/Format/FormatTestComments.cpp:2908 + Style.MaxEmptyLinesToKeep = 0; + EXPECT_EQ("int a; // comment\n" + "int ab; // comment\n" ---------------- Any reason why these are not verifyformats. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132131/new/ https://reviews.llvm.org/D132131 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits