Require ident in PatIdent to be a legal binding name

dtolnay · Apr 18, 2024 · 0a03036 · 0a03036
1 parent 6a28926
commit 0a03036
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/src/pat.rs b/src/pat.rs
@@ -470,7 +470,7 @@ pub(crate) mod parsing {
             attrs: Vec::new(),
             by_ref: input.parse()?,
             mutability: input.parse()?,
-            ident: input.call(Ident::parse_any)?,
+            ident: input.parse()?,
             subpat: {
                 if input.peek(Token![@]) {
                     let at_token: Token![@] = input.parse()?;

diff --git a/tests/test_pat.rs b/tests/test_pat.rs
@@ -9,14 +9,6 @@ use syn::parse::Parser;
 use syn::punctuated::Punctuated;
 use syn::{parse_quote, token, Item, Pat, PatTuple, Stmt, Token};
 
-#[test]
-fn test_pat_ident() {
-    match Pat::parse_single.parse2(quote!(self)).unwrap() {
-        Pat::Ident(_) => (),
-        value => panic!("expected PatIdent, got {:?}", value),
-    }
-}
-
 #[test]
 fn test_pat_path() {
     match Pat::parse_single.parse2(quote!(self::CONST)).unwrap() {