[PATCH] D119138: [clang-format] Further improve support for requires expressions

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 25 07:30:46 PST 2022


klimek added a comment.
Herald added a project: All.

Hey ho, sorry for the late comment here, but adding peekNextToken(n) is problematic, as this gets in the way of future changes we want to do to handle macros better.
Usually we want to use X = Tokens->getPosition() and FormatTok = Tokens->setPosition(X) pairs when doing look-ahead.
I did a quick attempt at fixing this, but ran into infinite loops later in the annotator :(


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119138



More information about the cfe-commits mailing list