[PATCH] D153798: [clang-format] Correctly annotate operator free function call

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 12:39:33 PDT 2023


HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added inline comments.


================
Comment at: clang/lib/Format/TokenAnnotator.cpp:316
+      // line can't be a declaration.
       bool OperatorCalledAsMemberFunction =
+          (Prev->Previous &&
----------------
Nuullll wrote:
> minor. Probably better to rename it to something like `IsOperatorCallSite`.
The Name should really be changes, since it's not limited to member functions anymore.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153798



More information about the cfe-commits mailing list