[PATCH] D63062: [clang-format] Added New Style Rule: OnePerLineBitFieldDecl

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 12 08:03:10 PDT 2019


MyDeveloperDay added a comment.

Nit: if you going for "OnePerLine" in the name rather than "Break" can you keep it consistent with `ConstructorInitializerAllOnOneLineOrOnePerLine` and put the type first and then the operation i.e.  `BitFieldOnePerLine` (this will help keep all BitField options together if there are ever more than one),  I also think either don't use `Decl` at all or use the full word `Declarations`  as in `AlignConsecutiveDeclarations` , `AlwaysBreakTemplateDeclarations` etc


Repository:
  rC Clang

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

https://reviews.llvm.org/D63062





More information about the cfe-commits mailing list