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

Reply via email to