rymiel added inline comments.
================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:3592-3601
case tok::kw_bool:
// bool is only allowed if it is directly followed by a paren for a cast:
// concept C = bool(...);
// and bool is the only type, all other types as cast must be inside a
// cast to bool an thus are handled by the other cases.
if (Tokens->peekNextToken()->isNot(tok::l_paren))
return;
----------------
rymiel wrote:
> owenpan wrote:
> > We should pass `true` to `peekNextToken()` on Line 3597 here, but removing
> > this entire case would have no effect on the existing unit test (Line 23693
> > in FormatTest.cpp). @HazardyKnusperkeks any idea?
> The peeking is there because of https://reviews.llvm.org/D134325, it has no
> format tests, only an annotator test
Also, I don't think that function is involved in concept declarations since
https://reviews.llvm.org/D140339
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142412/new/
https://reviews.llvm.org/D142412
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
