[PATCH] D90238: [clang-format] Added ReferenceAlignmentStyle option - (Update to D31635)

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 10 04:04:12 PDT 2021


MyDeveloperDay added a subscriber: HazardyKnusperkeks.
MyDeveloperDay added a comment.

@curdeius , @HazardyKnusperkeks any thoughts, personally I don't have any problems with this even if its not my personal preference.  @skirkovski if this is something you need we might need for you to take over the revision and bring it upto date and address the comments? is that something yo would be interesting in helping out with?

My personal preference is that someone who wants it should own it, as with all things we need volunteers. @catskul's last comment on D33029: [clang-format] add option for dangling parenthesis <https://reviews.llvm.org/D33029> was that they are busy, maybe they'd think the same here if someone wants to pick it up.

In my mind justification is that astyle supported seperate `align-pointer` and `align-reference`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90238



More information about the cfe-commits mailing list