[PATCH] D118991: [clang-format][docs] Fix incorrect 'clang-format 14' configuration options markers

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 5 03:27:40 PST 2022


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



More information about the cfe-commits mailing list