[PATCH] D143755: [clang-format] Add a space between an overloaded operator and '>'

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 20 02:47:24 PDT 2023


owenpan added a comment.

@kadircet thanks for reporting the crash and reverting the commit. I will fix it and reland the patch.



================
Comment at: clang/lib/Format/TokenAnnotator.cpp:1229
+          consumeToken();
+        assert(CurrentToken);
+        auto Previous = CurrentToken->getPreviousNonComment();
----------------
After `next()` or `consumeToken()`, `CurrentToken` would advance and could become null.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143755



More information about the cfe-commits mailing list