Rollup merge of #105505 - WaffleLapkin:yeet_unused_parens_lint, r=fee1-dead

matthiaskrgr · web-flow · commit 9e87dd907e1c · 2022-12-10T09:24:44.000+01:00
Don't warn about unused parens when they are used by yeet expr

Don't even get me started on how I've found this.
diff --git a/compiler/rustc_lint/src/unused.rs b/compiler/rustc_lint/src/unused.rs
@@ -617,7 +617,10 @@ trait UnusedDelimLint {
         lhs_needs_parens
             || (followed_by_block
                 && match &inner.kind {
-                    ExprKind::Ret(_) | ExprKind::Break(..) | ExprKind::Yield(..) => true,
+                    ExprKind::Ret(_)
+                    | ExprKind::Break(..)
+                    | ExprKind::Yield(..)
+                    | ExprKind::Yeet(..) => true,
                     ExprKind::Range(_lhs, Some(rhs), _limits) => {
                         matches!(rhs.kind, ExprKind::Block(..))
                     }
diff --git a/src/test/ui/lint/unused/issue-54538-unused-parens-lint.fixed b/src/test/ui/lint/unused/issue-54538-unused-parens-lint.fixed
@@ -1,6 +1,6 @@
 // run-rustfix
 
-#![feature(box_patterns, stmt_expr_attributes)]
+#![feature(box_patterns, stmt_expr_attributes, yeet_expr)]
 
 #![allow(
     dead_code,
@@ -25,6 +25,13 @@ fn _no_lint_attr() {
     let _x = #[allow(dead_code)] (1 + 2);
 }
 
+fn _no_lint_yeet() -> Result<(), ()> {
+    #[allow(unreachable_code)]
+    if (do yeet) {}
+
+    Ok(())
+}
+
 // Don't lint in these cases (#64106).
 fn or_patterns_no_lint() {
     match Box::new(0) {
diff --git a/src/test/ui/lint/unused/issue-54538-unused-parens-lint.rs b/src/test/ui/lint/unused/issue-54538-unused-parens-lint.rs
@@ -1,6 +1,6 @@
 // run-rustfix
 
-#![feature(box_patterns, stmt_expr_attributes)]
+#![feature(box_patterns, stmt_expr_attributes, yeet_expr)]
 
 #![allow(
     dead_code,
@@ -25,6 +25,13 @@ fn _no_lint_attr() {
     let _x = #[allow(dead_code)] (1 + 2);
 }
 
+fn _no_lint_yeet() -> Result<(), ()> {
+    #[allow(unreachable_code)]
+    if (do yeet) {}
+
+    Ok(())
+}
+
 // Don't lint in these cases (#64106).
 fn or_patterns_no_lint() {
     match Box::new(0) {
diff --git a/src/test/ui/lint/unused/issue-54538-unused-parens-lint.stderr b/src/test/ui/lint/unused/issue-54538-unused-parens-lint.stderr
@@ -76,7 +76,7 @@ LL +     let _ = |a: u8| 0;
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:49:12
+  --> $DIR/issue-54538-unused-parens-lint.rs:56:12
    |
 LL |     if let (0 | 1) = 0 {}
    |            ^     ^
@@ -88,7 +88,7 @@ LL +     if let 0 | 1 = 0 {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:50:13
+  --> $DIR/issue-54538-unused-parens-lint.rs:57:13
    |
 LL |     if let ((0 | 1),) = (0,) {}
    |             ^     ^
@@ -100,7 +100,7 @@ LL +     if let (0 | 1,) = (0,) {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:51:13
+  --> $DIR/issue-54538-unused-parens-lint.rs:58:13
    |
 LL |     if let [(0 | 1)] = [0] {}
    |             ^     ^
@@ -112,7 +112,7 @@ LL +     if let [0 | 1] = [0] {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:52:16
+  --> $DIR/issue-54538-unused-parens-lint.rs:59:16
    |
 LL |     if let 0 | (1 | 2) = 0 {}
    |                ^     ^
@@ -124,7 +124,7 @@ LL +     if let 0 | 1 | 2 = 0 {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:54:15
+  --> $DIR/issue-54538-unused-parens-lint.rs:61:15
    |
 LL |     if let TS((0 | 1)) = TS(0) {}
    |               ^     ^
@@ -136,7 +136,7 @@ LL +     if let TS(0 | 1) = TS(0) {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:56:20
+  --> $DIR/issue-54538-unused-parens-lint.rs:63:20
    |
 LL |     if let NS { f: (0 | 1) } = (NS { f: 0 }) {}
    |                    ^     ^
@@ -148,7 +148,7 @@ LL +     if let NS { f: 0 | 1 } = (NS { f: 0 }) {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:66:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:73:9
    |
 LL |         (_) => {}
    |         ^ ^
@@ -160,7 +160,7 @@ LL +         _ => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:67:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:74:9
    |
 LL |         (y) => {}
    |         ^ ^
@@ -172,7 +172,7 @@ LL +         y => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:68:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:75:9
    |
 LL |         (ref r) => {}
    |         ^     ^
@@ -184,7 +184,7 @@ LL +         ref r => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:69:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:76:9
    |
 LL |         (e @ 1...2) => {}
    |         ^         ^
@@ -196,7 +196,7 @@ LL +         e @ 1...2 => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:75:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:82:9
    |
 LL |         (e @ &(1...2)) => {}
    |         ^            ^
@@ -208,7 +208,7 @@ LL +         e @ &(1...2) => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:76:10
+  --> $DIR/issue-54538-unused-parens-lint.rs:83:10
    |
 LL |         &(_) => {}
    |          ^ ^
@@ -220,7 +220,7 @@ LL +         &_ => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:87:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:94:9
    |
 LL |         (_) => {}
    |         ^ ^
@@ -232,7 +232,7 @@ LL +         _ => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:88:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:95:9
    |
 LL |         (y) => {}
    |         ^ ^
@@ -244,7 +244,7 @@ LL +         y => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:89:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:96:9
    |
 LL |         (ref r) => {}
    |         ^     ^
@@ -256,7 +256,7 @@ LL +         ref r => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:90:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:97:9
    |
 LL |         (e @ 1..=2) => {}
    |         ^         ^
@@ -268,7 +268,7 @@ LL +         e @ 1..=2 => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:96:9
+  --> $DIR/issue-54538-unused-parens-lint.rs:103:9
    |
 LL |         (e @ &(1..=2)) => {}
    |         ^            ^
@@ -280,7 +280,7 @@ LL +         e @ &(1..=2) => {}
    |
 
 error: unnecessary parentheses around pattern
-  --> $DIR/issue-54538-unused-parens-lint.rs:97:10
+  --> $DIR/issue-54538-unused-parens-lint.rs:104:10
    |
 LL |         &(_) => {}
    |          ^ ^

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ LL + let _ = \|a: u8\| 0;`
`76`	`76`	`\|`
`77`	`77`
`78`	`78`	`error: unnecessary parentheses around pattern`
`79`		`- --> $DIR/issue-54538-unused-parens-lint.rs:49:12`
	`79`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:56:12`
`80`	`80`	`\|`
`81`	`81`	`LL \| if let (0 \| 1) = 0 {}`
`82`	`82`	`\| ^ ^`
`@@ -88,7 +88,7 @@ LL + if let 0 \| 1 = 0 {}`
`88`	`88`	`\|`
`89`	`89`
`90`	`90`	`error: unnecessary parentheses around pattern`
`91`		`- --> $DIR/issue-54538-unused-parens-lint.rs:50:13`
	`91`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:57:13`
`92`	`92`	`\|`
`93`	`93`	`LL \| if let ((0 \| 1),) = (0,) {}`
`94`	`94`	`\| ^ ^`
`@@ -100,7 +100,7 @@ LL + if let (0 \| 1,) = (0,) {}`
`100`	`100`	`\|`
`101`	`101`
`102`	`102`	`error: unnecessary parentheses around pattern`
`103`		`- --> $DIR/issue-54538-unused-parens-lint.rs:51:13`
	`103`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:58:13`
`104`	`104`	`\|`
`105`	`105`	`LL \| if let [(0 \| 1)] = [0] {}`
`106`	`106`	`\| ^ ^`
`@@ -112,7 +112,7 @@ LL + if let [0 \| 1] = [0] {}`
`112`	`112`	`\|`
`113`	`113`
`114`	`114`	`error: unnecessary parentheses around pattern`
`115`		`- --> $DIR/issue-54538-unused-parens-lint.rs:52:16`
	`115`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:59:16`
`116`	`116`	`\|`
`117`	`117`	`LL \| if let 0 \| (1 \| 2) = 0 {}`
`118`	`118`	`\| ^ ^`
`@@ -124,7 +124,7 @@ LL + if let 0 \| 1 \| 2 = 0 {}`
`124`	`124`	`\|`
`125`	`125`
`126`	`126`	`error: unnecessary parentheses around pattern`
`127`		`- --> $DIR/issue-54538-unused-parens-lint.rs:54:15`
	`127`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:61:15`
`128`	`128`	`\|`
`129`	`129`	`LL \| if let TS((0 \| 1)) = TS(0) {}`
`130`	`130`	`\| ^ ^`
`@@ -136,7 +136,7 @@ LL + if let TS(0 \| 1) = TS(0) {}`
`136`	`136`	`\|`
`137`	`137`
`138`	`138`	`error: unnecessary parentheses around pattern`
`139`		`- --> $DIR/issue-54538-unused-parens-lint.rs:56:20`
	`139`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:63:20`
`140`	`140`	`\|`
`141`	`141`	`LL \| if let NS { f: (0 \| 1) } = (NS { f: 0 }) {}`
`142`	`142`	`\| ^ ^`
`@@ -148,7 +148,7 @@ LL + if let NS { f: 0 \| 1 } = (NS { f: 0 }) {}`
`148`	`148`	`\|`
`149`	`149`
`150`	`150`	`error: unnecessary parentheses around pattern`
`151`		`- --> $DIR/issue-54538-unused-parens-lint.rs:66:9`
	`151`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:73:9`
`152`	`152`	`\|`
`153`	`153`	`LL \| (_) => {}`
`154`	`154`	`\| ^ ^`
`@@ -160,7 +160,7 @@ LL + _ => {}`
`160`	`160`	`\|`
`161`	`161`
`162`	`162`	`error: unnecessary parentheses around pattern`
`163`		`- --> $DIR/issue-54538-unused-parens-lint.rs:67:9`
	`163`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:74:9`
`164`	`164`	`\|`
`165`	`165`	`LL \| (y) => {}`
`166`	`166`	`\| ^ ^`
`@@ -172,7 +172,7 @@ LL + y => {}`
`172`	`172`	`\|`
`173`	`173`
`174`	`174`	`error: unnecessary parentheses around pattern`
`175`		`- --> $DIR/issue-54538-unused-parens-lint.rs:68:9`
	`175`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:75:9`
`176`	`176`	`\|`
`177`	`177`	`LL \| (ref r) => {}`
`178`	`178`	`\| ^ ^`
`@@ -184,7 +184,7 @@ LL + ref r => {}`
`184`	`184`	`\|`
`185`	`185`
`186`	`186`	`error: unnecessary parentheses around pattern`
`187`		`- --> $DIR/issue-54538-unused-parens-lint.rs:69:9`
	`187`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:76:9`
`188`	`188`	`\|`
`189`	`189`	`LL \| (e @ 1...2) => {}`
`190`	`190`	`\| ^ ^`
`@@ -196,7 +196,7 @@ LL + e @ 1...2 => {}`
`196`	`196`	`\|`
`197`	`197`
`198`	`198`	`error: unnecessary parentheses around pattern`
`199`		`- --> $DIR/issue-54538-unused-parens-lint.rs:75:9`
	`199`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:82:9`
`200`	`200`	`\|`
`201`	`201`	`LL \| (e @ &(1...2)) => {}`
`202`	`202`	`\| ^ ^`
`@@ -208,7 +208,7 @@ LL + e @ &(1...2) => {}`
`208`	`208`	`\|`
`209`	`209`
`210`	`210`	`error: unnecessary parentheses around pattern`
`211`		`- --> $DIR/issue-54538-unused-parens-lint.rs:76:10`
	`211`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:83:10`
`212`	`212`	`\|`
`213`	`213`	`LL \| &(_) => {}`
`214`	`214`	`\| ^ ^`
`@@ -220,7 +220,7 @@ LL + &_ => {}`
`220`	`220`	`\|`
`221`	`221`
`222`	`222`	`error: unnecessary parentheses around pattern`
`223`		`- --> $DIR/issue-54538-unused-parens-lint.rs:87:9`
	`223`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:94:9`
`224`	`224`	`\|`
`225`	`225`	`LL \| (_) => {}`
`226`	`226`	`\| ^ ^`
`@@ -232,7 +232,7 @@ LL + _ => {}`
`232`	`232`	`\|`
`233`	`233`
`234`	`234`	`error: unnecessary parentheses around pattern`
`235`		`- --> $DIR/issue-54538-unused-parens-lint.rs:88:9`
	`235`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:95:9`
`236`	`236`	`\|`
`237`	`237`	`LL \| (y) => {}`
`238`	`238`	`\| ^ ^`
`@@ -244,7 +244,7 @@ LL + y => {}`
`244`	`244`	`\|`
`245`	`245`
`246`	`246`	`error: unnecessary parentheses around pattern`
`247`		`- --> $DIR/issue-54538-unused-parens-lint.rs:89:9`
	`247`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:96:9`
`248`	`248`	`\|`
`249`	`249`	`LL \| (ref r) => {}`
`250`	`250`	`\| ^ ^`
`@@ -256,7 +256,7 @@ LL + ref r => {}`
`256`	`256`	`\|`
`257`	`257`
`258`	`258`	`error: unnecessary parentheses around pattern`
`259`		`- --> $DIR/issue-54538-unused-parens-lint.rs:90:9`
	`259`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:97:9`
`260`	`260`	`\|`
`261`	`261`	`LL \| (e @ 1..=2) => {}`
`262`	`262`	`\| ^ ^`
`@@ -268,7 +268,7 @@ LL + e @ 1..=2 => {}`
`268`	`268`	`\|`
`269`	`269`
`270`	`270`	`error: unnecessary parentheses around pattern`
`271`		`- --> $DIR/issue-54538-unused-parens-lint.rs:96:9`
	`271`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:103:9`
`272`	`272`	`\|`
`273`	`273`	`LL \| (e @ &(1..=2)) => {}`
`274`	`274`	`\| ^ ^`
`@@ -280,7 +280,7 @@ LL + e @ &(1..=2) => {}`
`280`	`280`	`\|`
`281`	`281`
`282`	`282`	`error: unnecessary parentheses around pattern`
`283`		`- --> $DIR/issue-54538-unused-parens-lint.rs:97:10`
	`283`	`+ --> $DIR/issue-54538-unused-parens-lint.rs:104:10`
`284`	`284`	`\|`
`285`	`285`	`LL \| &(_) => {}`
`286`	`286`	`\| ^ ^`