diff options
| author | Esteban Küber <esteban@kuber.com.ar> | 2025-01-23 18:40:41 +0000 |
|---|---|---|
| committer | Esteban Küber <esteban@kuber.com.ar> | 2025-02-06 22:28:44 +0000 |
| commit | 3815ed63ed0e1bfa2026e9a54b64a69e16f123ee (patch) | |
| tree | a8102a28fdd895322c4a5853eaf7c57d206e6f08 /tests/ui/for-loop-while/break.rs | |
| parent | 942db6782f4a28c55b0b75b38fd4394d0483390f (diff) | |
| download | rust-3815ed63ed0e1bfa2026e9a54b64a69e16f123ee.tar.gz rust-3815ed63ed0e1bfa2026e9a54b64a69e16f123ee.zip | |
Remove some unnecessary parens in `assert!` conditions
While working on #122661, some of these started triggering our "unnecessary parens" lints due to a change in the `assert!` desugaring. A cursory search identified a few more. Some of these have been carried from before 1.0, were a bulk rename from the previous name of `assert!` left them in that state. I went and removed as many of these unnecessary parens as possible in order to have fewer annoyances in the future if we make the lint smarter.
Diffstat (limited to 'tests/ui/for-loop-while/break.rs')
| -rw-r--r-- | tests/ui/for-loop-while/break.rs | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/ui/for-loop-while/break.rs b/tests/ui/for-loop-while/break.rs index 77774792262..442e07e148c 100644 --- a/tests/ui/for-loop-while/break.rs +++ b/tests/ui/for-loop-while/break.rs @@ -8,18 +8,18 @@ pub fn main() { assert_eq!(i, 20); let xs = [1, 2, 3, 4, 5, 6]; for x in &xs { - if *x == 3 { break; } assert!((*x <= 3)); + if *x == 3 { break; } assert!(*x <= 3); } i = 0; - while i < 10 { i += 1; if i % 2 == 0 { continue; } assert!((i % 2 != 0)); } + while i < 10 { i += 1; if i % 2 == 0 { continue; } assert!(i % 2 != 0); } i = 0; loop { - i += 1; if i % 2 == 0 { continue; } assert!((i % 2 != 0)); + i += 1; if i % 2 == 0 { continue; } assert!(i % 2 != 0); if i >= 10 { break; } } let ys = vec![1, 2, 3, 4, 5, 6]; for x in &ys { if *x % 2 == 0 { continue; } - assert!((*x % 2 != 0)); + assert!(*x % 2 != 0); } } |
