MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
This revision is now accepted and ready to land.

This looks good, I like to move towards one style, it was getting too 
confusing. Did you test this on a large code base at all? and maybe wait for 
one of the others to take a look.

I'll pull the patch to my local source and see if it makes any changes, but to 
be honest we only use the `BreakConstructorInitializersBeforeComma` option


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108752/new/

https://reviews.llvm.org/D108752

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to