[PATCH] D151047: [clang-format] Fix indentation for selective formatting.

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 22 01:42:29 PDT 2023


MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
This revision is now accepted and ready to land.

Thank you for the indepth explaination in https://github.com/llvm/llvm-project/issues/59178, that was really helpful for me trying to understand what the problem was.

I thik your explaination is clear, the reason why you are suggesting this change, and as it passed all the test, hopefully we can apply the "Beyonce Rule" on  https://github.com/llvm/llvm-project/issues/56352, I recommend we pull in @curdeius  as a review

I'm going to accept this, but I would ask one of the others gives it the all clear too, let us know if you have commit access and if not and want one of us to land it we'll need you name and email to attribute the change to you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151047



More information about the cfe-commits mailing list