[PATCH] D154184: [clang-format] Correctly annotate */&/&& in operator function calls

Emilia Kond via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 30 04:57:09 PDT 2023


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

Thanks, this probably makes more sense than what I did before



================
Comment at: clang/lib/Format/TokenAnnotator.cpp:3320
 
+  // Annotate */&/&& in `operator` function calls as binary operators.
+  for (const auto *Tok = Line.First; Tok; Tok = Tok->Next) {
----------------
I assume this extra fixup step means that the context of the call's parens won't be set to `IsExpression = true`? That won't cause any problems?


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

https://reviews.llvm.org/D154184



More information about the cfe-commits mailing list