owenca wrote:

Running `grep` in `clang/lib/Format/` gives the following (likely) missed ones:
```
$ grep -n isOneOf *.cpp | grep \!
ContinuationIndenter.cpp:371:  if (!Current.isOneOf(TT_BinaryOperator, 
tok::comma) &&
ContinuationIndenter.cpp:448:      !Current.isOneOf(tok::r_paren, 
tok::r_brace)) {
ContinuationIndenter.cpp:651:      !Current.isOneOf(tok::r_brace, 
tok::comment)) {
ContinuationIndenter.cpp:886:    if (!Previous || 
(!Previous->isOneOf(TT_FunctionDeclarationLParen,
ContinuationIndenter.cpp:965:      !P->isOneOf(TT_OverloadedOperator, 
TT_CtorInitializerComma) &&
ContinuationIndenter.cpp:995:  } else if (!Current.isOneOf(tok::comment, 
tok::caret) &&
ContinuationIndenter.cpp:1102:       !Current.isOneOf(Keywords.kw_async, 
Keywords.kw_function))) {
ContinuationIndenter.cpp:1594:      !PreviousNonComment->isOneOf(tok::r_brace, 
TT_CtorInitializerComma)) {
ContinuationIndenter.cpp:1761:  } else if (!Current.isOneOf(tok::comment, 
tok::identifier, tok::hash) &&
ContinuationIndenter.cpp:2060:      !Current.isOneOf(TT_DictLiteral, 
TT_ArrayInitializerLSquare) &&
Format.cpp:2438:        if (!Token->Optional || !Token->isOneOf(tok::l_brace, 
tok::r_brace))
FormatToken.cpp:111:  if (!LBrace || !LBrace->isOneOf(tok::l_brace, 
TT_ArrayInitializerLSquare) ||
FormatToken.cpp:180:      !Token->isOneOf(tok::l_brace, 
TT_ArrayInitializerLSquare)) {
FormatTokenLexer.cpp:736:  if (!RegexToken->isOneOf(tok::slash, 
tok::slashequal))
FormatTokenLexer.cpp:1044:  if (!HashToken->isOneOf(tok::hash, tok::hashhash))
MacroExpander.cpp:89:    if (!Current->isOneOf(tok::equal, tok::eof))
NamespaceEndCommentsFixer.cpp:73:    while (Tok && !Tok->isOneOf(tok::r_paren, 
tok::comma)) {
ObjCPropertyAttributeOrderFixer.cpp:64:    if (!Tok->isOneOf(tok::identifier, 
tok::kw_class)) {
QualifierAlignmentFixer.cpp:511:    if (!Previous || 
!Previous->isOneOf(tok::kw_struct, tok::kw_class)) {
SortJavaScriptImports.cpp:442:    if (!Current || 
!Current->isOneOf(Keywords.kw_import, tok::kw_export))
SortJavaScriptImports.cpp:573:      if (!Current->isOneOf(tok::r_brace, 
tok::comma))
TokenAnnotator.cpp:206:              (!Next || !Next->isOneOf(tok::l_paren, 
tok::l_brace))) {
TokenAnnotator.cpp:254:      if (Prev.isOneOf(tok::question, tok::colon) && 
!Style.isProto())
TokenAnnotator.cpp:707:        !CurrentToken->isOneOf(tok::l_brace, 
tok::r_square) &&
TokenAnnotator.cpp:1342:             !Line.First->isOneOf(tok::kw_enum, 
tok::kw_case)) ||
TokenAnnotator.cpp:1419:             Token && !Token->isOneOf(tok::semi, 
tok::l_paren);
TokenAnnotator.cpp:1570:          !Tok->isOneOf(TT_TypeDeclarationParen, 
TT_RequiresExpressionLParen)) {
TokenAnnotator.cpp:1736:            (!Next || !Next->isOneOf(tok::identifier, 
tok::string_literal) ||
TokenAnnotator.cpp:1737:             !Next->Next || 
!Next->Next->isOneOf(tok::colon, tok::question))) {
TokenAnnotator.cpp:1804:                   (!Next || !Next->isOneOf(tok::colon, 
tok::l_brace))) {
TokenAnnotator.cpp:2238:          return 
!Current.Previous->Previous->isOneOf(tok::kw_typename,
TokenAnnotator.cpp:2274:             !Previous->Previous->isOneOf(tok::comma, 
tok::semi);
TokenAnnotator.cpp:2438:               
!Current.Previous->isOneOf(tok::kw_operator, tok::identifier)) {
TokenAnnotator.cpp:2666:             
!PreviousNotConst->MatchingParen->Previous->isOneOf(
TokenAnnotator.cpp:2926:        !AfterRParen->Next->isOneOf(tok::identifier, 
tok::numeric_constant)) {
TokenAnnotator.cpp:3137:      return !token || token->isOneOf(tok::amp, 
tok::period, tok::arrow,
TokenAnnotator.cpp:3865:        !Previous.isOneOf(tok::kw_return, 
tok::kw_co_return)) {
TokenAnnotator.cpp:4583:      !Right.isOneOf(tok::semi, tok::r_paren)) {
TokenAnnotator.cpp:4660:    return !Left.isOneOf(TT_PointerOrReference, 
tok::l_paren) &&
TokenAnnotator.cpp:4733:    return !BeforeLeft->isOneOf(tok::l_paren, 
tok::l_square);
TokenAnnotator.cpp:4898:    if (!BeforeLeft || 
!BeforeLeft->isOneOf(tok::period, tok::arrow)) {
TokenAnnotator.cpp:5303:         !Right.isOneOf(tok::r_paren, tok::semi)) ||
TokenAnnotator.cpp:5352:        !(Left.isOneOf(Keywords.kw_assign, 
Keywords.kw_unique) ||
TokenAnnotator.cpp:5571:    return !Left.isOneOf(tok::amp, tok::ampamp) ||
TokenAnnotator.cpp:5621:         !Tok.isOneOf(TT_ObjCBlockLBrace, 
TT_DictLiteral);
TokenAnnotator.cpp:5690:        !Line.First->isOneOf(Keywords.kw_var, 
Keywords.kw_let)) {
TokenAnnotator.cpp:5835:    return !Left.isOneOf(BK_BracedInit, 
TT_CtorInitializerColon) &&
TokenAnnotator.cpp:5877:      return !Right.isOneOf(tok::semi, tok::l_brace);
TokenAnnotator.cpp:6004:      !Right.isOneOf(TT_LeadingJavaAnnotation, 
tok::l_paren) &&
TokenAnnotator.cpp:6382:  if (Left.is(tok::equal) && 
!Right.isOneOf(tok::kw_default, tok::kw_delete) &&
TokenAnnotator.cpp:6452:      !Left.isOneOf(tok::arrowstar, tok::lessless) &&
UnwrappedLineFormatter.cpp:780:    if (!Style.isJava() && 
Line.First->isOneOf(tok::at, tok::minus, tok::plus))
UnwrappedLineParser.cpp:408:            
!OpeningBrace->isOneOf(TT_ControlStatementLBrace, TT_ElseLBrace)) {
UnwrappedLineParser.cpp:430:      if (!Next->isOneOf(tok::colon, tok::arrow)) {
UnwrappedLineParser.cpp:1160:      if 
(!Line.Tokens.front().Tok->isOneOf(tok::comment, tok::hash)) {
UnwrappedLineParser.cpp:1259:          !FormatTok->isOneOf(
UnwrappedLineParser.cpp:1325:      !Tok->isOneOf(tok::kw_register, 
tok::kw_struct, tok::kw_union)) {
UnwrappedLineParser.cpp:1348:      !Token->isOneOf(tok::colon, tok::less, 
tok::string_literal)) {
UnwrappedLineParser.cpp:1360:      while (!FormatTok->isOneOf(tok::semi, 
tok::greater) && !eof()) {
UnwrappedLineParser.cpp:2392:        !Previous->isOneOf(tok::kw_return, 
tok::kw_co_await, tok::kw_co_yield,
UnwrappedLineParser.cpp:2403:      if (!BeforeRParen || 
!BeforeRParen->isOneOf(tok::greater, tok::r_paren))
UnwrappedLineParser.cpp:2452:      while (!FormatTok->isOneOf(tok::l_brace, 
tok::semi) && !eof())
UnwrappedLineParser.cpp:3110:    if (!(FormatTok->isOneOf(tok::kw_catch, 
Keywords.kw___except,
UnwrappedLineParser.cpp:3292:                          
!FormatTok->isOneOf(tok::kw_for, tok::kw_while);
UnwrappedLineParser.cpp:4341:  if (!IsImport && 
!FormatTok->isOneOf(tok::l_brace, tok::star) &&
```

https://github.com/llvm/llvm-project/pull/161021
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to