Skip to content

Commit 6365093

Browse files
committed
Revert "Ignore using destructuring assignment to wildcard pattern"
This reverts commit 13cd0a2.
1 parent 66ec197 commit 6365093

File tree

10 files changed

+39
-31
lines changed

10 files changed

+39
-31
lines changed

benches/rust.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
#![recursion_limit = "1024"]
88
#![allow(
99
clippy::cast_lossless,
10+
clippy::let_underscore_untyped,
1011
clippy::manual_let_else,
1112
clippy::match_like_matches_macro,
1213
clippy::uninlined_format_args,
@@ -98,7 +99,7 @@ mod librustc_parse {
9899
#[cfg(not(syn_only))]
99100
mod read_from_disk {
100101
pub fn bench(content: &str) -> Result<(), ()> {
101-
_ = content;
102+
let _ = content;
102103
Ok(())
103104
}
104105
}

examples/dump-syntax/src/main.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,14 @@ impl Display for Error {
5555

5656
fn main() {
5757
if let Err(error) = try_main() {
58-
_ = writeln!(io::stderr(), "{}", error);
58+
let _ = writeln!(io::stderr(), "{}", error);
5959
process::exit(1);
6060
}
6161
}
6262

6363
fn try_main() -> Result<(), Error> {
6464
let mut args = env::args_os();
65-
_ = args.next(); // executable name
65+
let _ = args.next(); // executable name
6666

6767
let filepath = match (args.next(), args.next()) {
6868
(Some(arg), None) => PathBuf::from(arg),

fuzz/fuzz_targets/create_token_buffer.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ fn immediate_fail(_input: ParseStream) -> syn::Result<()> {
1212
fuzz_target!(|data: &[u8]| {
1313
if data.len() < 300 {
1414
if let Ok(string) = str::from_utf8(data) {
15-
_ = immediate_fail.parse_str(string);
15+
let _ = immediate_fail.parse_str(string);
1616
}
1717
}
1818
});

fuzz/fuzz_targets/parse_file.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ use std::str;
66
fuzz_target!(|data: &[u8]| {
77
if data.len() < 300 {
88
if let Ok(string) = str::from_utf8(data) {
9-
_ = syn::parse_file(string);
9+
let _ = syn::parse_file(string);
1010
}
1111
}
1212
});

tests/crates/main.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
// | jq '"https://static.crates.io/crates/" + .name + "/" + .name + "-" + .vers + ".crate"' -r \
1111
// | xargs -P10 -n100 wget -nc
1212

13-
#![allow(clippy::uninlined_format_args)]
13+
#![allow(clippy::let_underscore_untyped, clippy::uninlined_format_args)]
1414

1515
use anyhow::{ensure, Result};
1616
use flate2::read::GzDecoder;
@@ -78,7 +78,7 @@ fn parse(path: &Path, oversize_count: &AtomicUsize) -> Result<()> {
7878
if entry.read_to_string(&mut contents).is_err() {
7979
break;
8080
}
81-
_ = syn::parse_file(&contents);
81+
let _ = syn::parse_file(&contents);
8282
}
8383
Ok(())
8484
}

tests/features/macro/lib.rs

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
1-
#![allow(clippy::toplevel_ref_arg, clippy::uninlined_format_args)]
1+
#![allow(
2+
clippy::let_underscore_untyped,
3+
clippy::toplevel_ref_arg,
4+
clippy::uninlined_format_args
5+
)]
26

37
use proc_macro::TokenStream;
48
use std::io::Write;
@@ -23,31 +27,31 @@ pub fn check(_input: TokenStream) -> TokenStream {
2327

2428
if cfg!(debug_assertions) {
2529
let yellow = ColorSpec::new().set_fg(Some(Color::Yellow)).clone();
26-
_ = writeln!(stderr);
27-
_ = stderr.set_color(yellow.clone().set_bold(true));
28-
_ = write!(stderr, "NOTE");
30+
let _ = writeln!(stderr);
31+
let _ = stderr.set_color(yellow.clone().set_bold(true));
32+
let _ = write!(stderr, "NOTE");
2933
for line in DEBUG_NOTE.lines() {
30-
_ = stderr.set_color(&yellow);
31-
_ = writeln!(stderr, "{}", line);
34+
let _ = stderr.set_color(&yellow);
35+
let _ = writeln!(stderr, "{}", line);
3236
}
33-
_ = stderr.reset();
34-
_ = writeln!(stderr);
37+
let _ = stderr.reset();
38+
let _ = writeln!(stderr);
3539
needs_newline = false;
3640
}
3741

3842
if cfg!(not(feature = "all-features")) {
3943
let red = ColorSpec::new().set_fg(Some(Color::Red)).clone();
4044
if needs_newline {
41-
_ = writeln!(stderr);
45+
let _ = writeln!(stderr);
4246
}
43-
_ = stderr.set_color(red.clone().set_bold(true));
44-
_ = write!(stderr, "ERROR");
47+
let _ = stderr.set_color(red.clone().set_bold(true));
48+
let _ = write!(stderr, "ERROR");
4549
for line in FEATURES_ERROR.lines() {
46-
_ = stderr.set_color(&red);
47-
_ = writeln!(stderr, "{}", line);
50+
let _ = stderr.set_color(&red);
51+
let _ = writeln!(stderr, "{}", line);
4852
}
49-
_ = stderr.reset();
50-
_ = writeln!(stderr);
53+
let _ = stderr.reset();
54+
let _ = writeln!(stderr);
5155
process::exit(1);
5256
}
5357

tests/regression.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#![allow(clippy::uninlined_format_args)]
1+
#![allow(clippy::let_underscore_untyped, clippy::uninlined_format_args)]
22

33
mod regression {
44
automod::dir!("tests/regression");

tests/regression/issue1108.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#[test]
22
fn issue1108() {
33
let data = "impl<x<>>::x for";
4-
_ = syn::parse_file(data);
4+
let _ = syn::parse_file(data);
55
}

tests/test_parse_stream.rs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1+
#![allow(clippy::let_underscore_untyped)]
2+
13
use syn::ext::IdentExt;
24
use syn::parse::ParseStream;
35
use syn::{Ident, Token};
46

57
#[test]
68
fn test_peek() {
7-
_ = |input: ParseStream| {
8-
_ = input.peek(Ident);
9-
_ = input.peek(Ident::peek_any);
10-
_ = input.peek(Token![::]);
9+
let _ = |input: ParseStream| {
10+
let _ = input.peek(Ident);
11+
let _ = input.peek(Ident::peek_any);
12+
let _ = input.peek(Token![::]);
1113
};
1214
}

tests/test_precedence.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
#![feature(rustc_private)]
55
#![allow(
66
clippy::explicit_deref_methods,
7+
clippy::let_underscore_untyped,
78
clippy::manual_assert,
89
clippy::manual_let_else,
910
clippy::match_like_matches_macro,
@@ -317,15 +318,15 @@ fn librustc_brackets(mut librustc_expr: P<ast::Expr>) -> Option<P<ast::Expr>> {
317318
// types yet. We'll look into comparing those in the future. For now
318319
// focus on expressions appearing in other places.
319320
fn visit_pat(&mut self, pat: &mut P<Pat>) {
320-
_ = pat;
321+
let _ = pat;
321322
}
322323

323324
fn visit_ty(&mut self, ty: &mut P<Ty>) {
324-
_ = ty;
325+
let _ = ty;
325326
}
326327

327328
fn visit_attribute(&mut self, attr: &mut Attribute) {
328-
_ = attr;
329+
let _ = attr;
329330
}
330331
}
331332

0 commit comments

Comments
 (0)