Skip to content

Commit 55aa9d2

Browse files
committed
Add run-rustfix for range_plus_minus_one tests
1 parent 18fb1d9 commit 55aa9d2

File tree

3 files changed

+49
-8
lines changed

3 files changed

+49
-8
lines changed

tests/ui/range_plus_minus_one.fixed

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
// run-rustfix
2+
3+
#![allow(unused_parens)]
4+
5+
fn f() -> usize {
6+
42
7+
}
8+
9+
#[warn(clippy::range_plus_one)]
10+
fn main() {
11+
for _ in 0..2 {}
12+
for _ in 0..=2 {}
13+
14+
for _ in 0..=3 {}
15+
for _ in 0..=3 + 1 {}
16+
17+
for _ in 0..=5 {}
18+
for _ in 0..=1 + 5 {}
19+
20+
for _ in 1..=1 {}
21+
for _ in 1..=1 + 1 {}
22+
23+
for _ in 0..13 + 13 {}
24+
for _ in 0..=13 - 7 {}
25+
26+
for _ in 0..=f() {}
27+
for _ in 0..=(1 + f()) {}
28+
29+
let _ = ..11 - 1;
30+
let _ = ..11;
31+
let _ = ..11;
32+
let _ = (1..=11);
33+
let _ = ((f() + 1)..=f());
34+
35+
let mut vec: Vec<()> = std::vec::Vec::new();
36+
vec.drain(..);
37+
}

tests/ui/range_plus_minus_one.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
// run-rustfix
2+
3+
#![allow(unused_parens)]
4+
15
fn f() -> usize {
26
42
37
}

tests/ui/range_plus_minus_one.stderr

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,51 +1,51 @@
11
error: an inclusive range would be more readable
2-
--> $DIR/range_plus_minus_one.rs:10:14
2+
--> $DIR/range_plus_minus_one.rs:14:14
33
|
44
LL | for _ in 0..3 + 1 {}
55
| ^^^^^^^^ help: use: `0..=3`
66
|
77
= note: `-D clippy::range-plus-one` implied by `-D warnings`
88

99
error: an inclusive range would be more readable
10-
--> $DIR/range_plus_minus_one.rs:13:14
10+
--> $DIR/range_plus_minus_one.rs:17:14
1111
|
1212
LL | for _ in 0..1 + 5 {}
1313
| ^^^^^^^^ help: use: `0..=5`
1414

1515
error: an inclusive range would be more readable
16-
--> $DIR/range_plus_minus_one.rs:16:14
16+
--> $DIR/range_plus_minus_one.rs:20:14
1717
|
1818
LL | for _ in 1..1 + 1 {}
1919
| ^^^^^^^^ help: use: `1..=1`
2020

2121
error: an inclusive range would be more readable
22-
--> $DIR/range_plus_minus_one.rs:22:14
22+
--> $DIR/range_plus_minus_one.rs:26:14
2323
|
2424
LL | for _ in 0..(1 + f()) {}
2525
| ^^^^^^^^^^^^ help: use: `0..=f()`
2626

2727
error: an exclusive range would be more readable
28-
--> $DIR/range_plus_minus_one.rs:26:13
28+
--> $DIR/range_plus_minus_one.rs:30:13
2929
|
3030
LL | let _ = ..=11 - 1;
3131
| ^^^^^^^^^ help: use: `..11`
3232
|
3333
= note: `-D clippy::range-minus-one` implied by `-D warnings`
3434

3535
error: an exclusive range would be more readable
36-
--> $DIR/range_plus_minus_one.rs:27:13
36+
--> $DIR/range_plus_minus_one.rs:31:13
3737
|
3838
LL | let _ = ..=(11 - 1);
3939
| ^^^^^^^^^^^ help: use: `..11`
4040

4141
error: an inclusive range would be more readable
42-
--> $DIR/range_plus_minus_one.rs:28:13
42+
--> $DIR/range_plus_minus_one.rs:32:13
4343
|
4444
LL | let _ = (1..11 + 1);
4545
| ^^^^^^^^^^^ help: use: `(1..=11)`
4646

4747
error: an inclusive range would be more readable
48-
--> $DIR/range_plus_minus_one.rs:29:13
48+
--> $DIR/range_plus_minus_one.rs:33:13
4949
|
5050
LL | let _ = (f() + 1)..(f() + 1);
5151
| ^^^^^^^^^^^^^^^^^^^^ help: use: `((f() + 1)..=f())`

0 commit comments

Comments
 (0)