[PATCH] D141959: [clang-format] Fix inconsistent identification of operator&

David K Turner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 17 16:09:49 PST 2023


dkt01 marked an inline comment as done.
dkt01 added inline comments.


================
Comment at: clang/lib/Format/TokenAnnotator.cpp:1205
     case tok::r_brace:
+      if (Scopes.size() > 1)
+        Scopes.pop_back();
----------------
HazardyKnusperkeks wrote:
> How does this happen? The only `push` I can see is before `parseBrace`, where a `pop` is following directly after.
I previously had a double pop for some braces, but now the checks are no longer necessary as pushes and pops match.


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

https://reviews.llvm.org/D141959



More information about the cfe-commits mailing list