[PATCH] D153641: [clang-format] Preserve AmpAmpTokenType in nested parentheses
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Jun 24 03:58:55 PDT 2023
HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.
This revision is now accepted and ready to land.
In D153641#4444465 <https://reviews.llvm.org/D153641#4444465>, @rymiel wrote:
> @HazardyKnusperkeks I'm not sure why it didn't recurse already, given that you even documented that it doesn't, but I chose to trust in the Beyoncé rule.
Most likely I didn't thought about what you now fixed and wanted to limit the scope of my change. I'm trying to think of something you may broke, but I can't.
Declaring functions should not be possible, if one already set the binary operator, right? Lambdas would be possible, but they should be handled correctly, but I think you could add something with `[](T&&){}(t)` in a clause, to show that it is correctly parsed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153641/new/
https://reviews.llvm.org/D153641
More information about the cfe-commits
mailing list