[PATCH] D119682: [clang-format][docs] Fix incorrect 'clang-format 13' configuration options markers
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 18 09:19:32 PST 2022
HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.
>> That's not the first time we renamed something. And most likely not the last time.
>
> But that doesn't mean we can't add "Previously known as IndentRequires", does it? :)
Of course! I just wanted to say that we can and should create precedence here.
================
Comment at: clang/include/clang/Format/Format.h:2540-2541
/// }
/// \endcode
- /// \version 13
+ /// \version 15
bool IndentRequiresClause;
----------------
kuzkry wrote:
> MyDeveloperDay wrote:
> > HazardyKnusperkeks wrote:
> > >
> > I like that suggestion..
> Sorry, I didn't notice this inline comment before.
>
> @HazardyKnusperkeks, I put it in a slightly different place, i.e. above the example code as Sphinx complained about this particular placement.
Fine by me. I have absolutely no idea about this sphinx or `.rst` stuff.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119682/new/
https://reviews.llvm.org/D119682
More information about the cfe-commits
mailing list