[clang] [Clang][P1061] Add stuctured binding packs (PR #121417)

Younan Zhang via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 2 01:40:47 PST 2025


================
@@ -7331,18 +7332,33 @@ void Parser::ParseDecompositionDeclarator(Declarator &D) {
           Diag(Tok, diag::err_expected_comma_or_rsquare);
         }
 
-        SkipUntil(tok::r_square, tok::comma, tok::identifier,
-                  StopAtSemi | StopBeforeMatch);
+        // I don't know why this skipping was here
+        // SkipUntil(tok::r_square, tok::comma, tok::identifier,
+        //          StopAtSemi | StopBeforeMatch);
         if (Tok.is(tok::comma))
           ConsumeToken();
-        else if (Tok.isNot(tok::identifier))
+        else if (Tok.is(tok::r_square))
           break;
       }
     }
 
     if (isCXX11AttributeSpecifier())
       DiagnoseAndSkipCXX11Attributes();
 
+    SourceLocation EllipsisLoc = {};
----------------
zyn0217 wrote:

nit: You don't have to initialize a SourceLocation explicitly.

https://github.com/llvm/llvm-project/pull/121417


More information about the cfe-commits mailing list