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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
