[PATCH] D94661: [clang-format] [PR19056] Add support for access modifiers indentation

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 4 06:38:47 PST 2021


curdeius added a comment.

Don't forget to mark comments as Done when... they're done :).



================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2039
 
+**IndentAccessModifiers** (``bool``)
+  Makes an indentation level for record (``class``, ``struct``, ``union``)
----------------
Budovi wrote:
> Budovi wrote:
> > curdeius wrote:
> > > curdeius wrote:
> > > > The name `IndentAccessModifiers` is misleading for me. Access modifiers are public, protected, private.
> > > Hmm, actually that's the description that is misleading.
> > Hm. Will go through the bug report again to see how people describe it. I'm open to suggestions, but I'll need to fix it anyway because I see a grammar mistake.
> > 
> > The current way I tried to go about it is that, without this option, modifiers don't have their own indentation level. Without the offset given by the `AccessModifierOffset` they would just be flush with the record members.
> > 
> > The introduced option creates a level designated for the access modifiers by shifting the members further to the right.
> Hopefully the description is better this time.
Much better indeed! Thanks!


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

https://reviews.llvm.org/D94661



More information about the cfe-commits mailing list