[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