HazardyKnusperkeks added a comment. In D119599#3336936 <https://reviews.llvm.org/D119599#3336936>, @curdeius wrote:
> Nice job! Please let us a bit of time to review that. > Also, I think it would be good to get a reviewer that knows well the yaml > parts. Or even split it to a separate revision. > And this part needs tests too. Yeah really thanks for doing this. But please split it up. ================ Comment at: clang/lib/Format/Format.cpp:1161 + LLVMStyle.AlignConsecutiveAssignments = { + /*.Enabled=*/false, /*.AcrossEmptyLines=*/false, + /*.AcrossComments=*/false, /*.AlignCompound=*/false, ---------------- Without the . 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