[PATCH] D119682: [clang-format][docs] Fix incorrect 'clang-format 13' configuration options markers
Krystian Kuzniarek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 18 07:35:04 PST 2022
kuzkry updated this revision to Diff 409935.
kuzkry added a comment.
Review fix: added note about IndentRequires. The generated HTML looks like this:
F22153448: IndentRequires note.png <https://reviews.llvm.org/F22153448>
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119682/new/
https://reviews.llvm.org/D119682
Files:
clang/docs/ClangFormatStyleOptions.rst
clang/include/clang/Format/Format.h
Index: clang/include/clang/Format/Format.h
===================================================================
--- clang/include/clang/Format/Format.h
+++ clang/include/clang/Format/Format.h
@@ -1791,7 +1791,7 @@
};
/// The concept declaration style to use.
- /// \version 13
+ /// \version 12
BreakBeforeConceptDeclarationsStyle BreakBeforeConceptDeclarations;
/// If ``true``, ternary operators will be placed after line breaks.
@@ -2185,7 +2185,7 @@
};
/// Defines in which cases to put empty line before access modifiers.
- /// \version 13
+ /// \version 12
EmptyLineBeforeAccessModifierStyle EmptyLineBeforeAccessModifier;
/// If ``true``, clang-format detects whether function calls and
@@ -2523,6 +2523,8 @@
/// Indent the requires clause in a template. This only applies when
/// ``RequiresClausePosition`` is ``OwnLine``, or ``WithFollowing``.
+ ///
+ /// In clang-format 13 and 14 it was named ``IndentRequires``.
/// \code
/// true:
/// template <typename It>
@@ -2538,7 +2540,7 @@
/// //....
/// }
/// \endcode
- /// \version 13
+ /// \version 15
bool IndentRequiresClause;
/// The number of columns to use for indentation.
Index: clang/docs/ClangFormatStyleOptions.rst
===================================================================
--- clang/docs/ClangFormatStyleOptions.rst
+++ clang/docs/ClangFormatStyleOptions.rst
@@ -1988,7 +1988,7 @@
-**BreakBeforeConceptDeclarations** (``BreakBeforeConceptDeclarationsStyle``) :versionbadge:`clang-format 13`
+**BreakBeforeConceptDeclarations** (``BreakBeforeConceptDeclarationsStyle``) :versionbadge:`clang-format 12`
The concept declaration style to use.
Possible values:
@@ -2278,7 +2278,7 @@
-**EmptyLineBeforeAccessModifier** (``EmptyLineBeforeAccessModifierStyle``) :versionbadge:`clang-format 13`
+**EmptyLineBeforeAccessModifier** (``EmptyLineBeforeAccessModifierStyle``) :versionbadge:`clang-format 12`
Defines in which cases to put empty line before access modifiers.
Possible values:
@@ -2706,10 +2706,12 @@
-**IndentRequiresClause** (``Boolean``) :versionbadge:`clang-format 13`
+**IndentRequiresClause** (``Boolean``) :versionbadge:`clang-format 15`
Indent the requires clause in a template. This only applies when
``RequiresClausePosition`` is ``OwnLine``, or ``WithFollowing``.
+ In clang-format 13 and 14 it was named ``IndentRequires``.
+
.. code-block:: c++
true:
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D119682.409935.patch
Type: text/x-patch
Size: 2483 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220218/df2f4298/attachment-0001.bin>
More information about the cfe-commits
mailing list