HazardyKnusperkeks wrote:

> Running `grep` in `clang/lib/Format/` gives the following (likely) missed 
> ones:
> 
> ```
> $ grep -nw 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) &&
> ```

Don't know anymore what my search was, but I'm a bit embarrassed to have missed 
that much...

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