[PATCH] D139801: [clang-format] Adds 'friend' to QualifierOrder.

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 12 12:56:48 PST 2022


HazardyKnusperkeks added a comment.

In D139801#3989284 <https://reviews.llvm.org/D139801#3989284>, @red1bluelost wrote:

> In D139801#3988790 <https://reviews.llvm.org/D139801#3988790>, @HazardyKnusperkeks wrote:
>
>> If only the first, I'd say we do that without adding this option.
>
> I'll work on doing this without adding the option. (Just reread and noticed you had answered what I asked in my previous comment.)

It should be part of the qualifieralignmentfixer, we don't want to move the tokens when someone doesn't have it turned on. My idea would be hardcoded `friend` as first entry, regardless of the chosen sequence.
But others may have a different opinion on that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139801/new/

https://reviews.llvm.org/D139801



More information about the cfe-commits mailing list