[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