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

[Jakub Budiský via Phabricator via cfe-commits]

Budovi added inline comments.


================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2039
 
+**IndentAccessModifiers** (``bool``)
+  Makes an indentation level for record (``class``, ``struct``, ``union``)
----------------
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.


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

https://reviews.llvm.org/D94661