[PATCH] D129940: [clang-format] Fix misannotation of colon in presence of requires clause
Marek Kurdej via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jul 17 00:54:36 PDT 2022
curdeius accepted this revision.
curdeius added a comment.
Ok for me if it's OK for Owen.
================
Comment at: clang/unittests/Format/TokenAnnotatorTest.cpp:820-833
+ ASSERT_EQ(BaseTokens.size(), NumberOfBaseTokens) << BaseTokens;
+ ASSERT_EQ(ConstrainedTokens.size(),
+ NumberOfBaseTokens + NumberOfAdditionalRequiresClauseTokens)
+ << ConstrainedTokens;
+
+ for (auto I = 0u; I < NumberOfBaseTokens; ++I) {
+ if (I < NumberOfTokensBeforeRequires) {
----------------
owenpan wrote:
> Can you make it a function or lambda?
:+1:
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129940/new/
https://reviews.llvm.org/D129940
More information about the cfe-commits
mailing list