curdeius added a comment. That looks like a strange style option to me. Is there any bigger codebase formatting the code this way?
================ Comment at: clang/docs/ClangFormatStyleOptions.rst:2039 +**IndentAccessModifiers** (``bool``) + Makes an indentation level for record (``class``, ``struct``, ``union``) ---------------- The name `IndentAccessModifiers` is misleading for me. Access modifiers are public, protected, private. ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:2039 +**IndentAccessModifiers** (``bool``) + Makes an indentation level for record (``class``, ``struct``, ``union``) ---------------- curdeius wrote: > The name `IndentAccessModifiers` is misleading for me. Access modifiers are > public, protected, private. Hmm, actually that's the description that is misleading. ================ Comment at: clang/unittests/Format/FormatTest.cpp:17868 + FormatStyle Style = getLLVMStyle(); + Style.AccessModifierOffset = 4; + Style.AllowShortEnumsOnASingleLine = false; ---------------- Why are those non-related style options here? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94661/new/ https://reviews.llvm.org/D94661 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
