MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land.
Thank you for this. LGTM (I double-checked a few and they all look good to me!) ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:2212 -**EmptyLineAfterAccessModifier** (``EmptyLineAfterAccessModifierStyle``) :versionbadge:`clang-format 14` +**EmptyLineAfterAccessModifier** (``EmptyLineAfterAccessModifierStyle``) :versionbadge:`clang-format 13` Defines when to put an empty line after access modifiers. ---------------- agreed its here https://releases.llvm.org/13.0.0/tools/clang/docs/ClangFormatStyleOptions.html ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:2378 -**IfMacros** (``List of Strings``) :versionbadge:`clang-format 14` +**IfMacros** (``List of Strings``) :versionbadge:`clang-format 13` A vector of macros that should be interpreted as conditionals ---------------- Agreed https://releases.llvm.org/13.0.0/tools/clang/docs/ClangFormatStyleOptions.html ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:3130 -**PPIndentWidth** (``Integer``) :versionbadge:`clang-format 14` +**PPIndentWidth** (``Integer``) :versionbadge:`clang-format 13` The number of columns to use for indentation of preprocessor statements. ---------------- Agreed, https://reviews.llvm.org/D103286 {F21991078} ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:3374 -**ReferenceAlignment** (``ReferenceAlignmentStyle``) :versionbadge:`clang-format 14` +**ReferenceAlignment** (``ReferenceAlignmentStyle``) :versionbadge:`clang-format 13` Reference alignment style (overrides ``PointerAlignment`` for ---------------- Agreed https://reviews.llvm.org/D104096 {F21991083} ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:3538 -**ShortNamespaceLines** (``Unsigned``) :versionbadge:`clang-format 14` +**ShortNamespaceLines** (``Unsigned``) :versionbadge:`clang-format 13` The maximal number of unwrapped lines that a short namespace spans. ---------------- Agreed https://reviews.llvm.org/D87587 {F21991090} Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118991/new/ https://reviews.llvm.org/D118991 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
