[PATCH] D108752: [clang-format] Group options that pack constructor initializers

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 26 05:58:56 PDT 2021


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



More information about the cfe-commits mailing list