Skip to content

Commit 3b4c263

Browse files
committed
add uitest for indexing in unnecessary const items
1 parent 31f7943 commit 3b4c263

File tree

3 files changed

+43
-3
lines changed

3 files changed

+43
-3
lines changed

tests/ui/unnecessary_operation.fixed

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ fn get_number() -> i32 {
4343
0
4444
}
4545

46-
fn get_usize() -> usize {
46+
const fn get_usize() -> usize {
4747
0
4848
}
4949
fn get_struct() -> Struct {
@@ -113,4 +113,15 @@ fn main() {
113113
'label: {
114114
break 'label
115115
};
116+
let () = const {
117+
assert!([42, 55].len() > get_usize());
118+
};
119+
}
120+
121+
const _: () = {
122+
assert!([42, 55].len() > get_usize());
123+
};
124+
125+
const fn foo() {
126+
assert!([42, 55].len() > get_usize());
116127
}

tests/ui/unnecessary_operation.rs

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ fn get_number() -> i32 {
4343
0
4444
}
4545

46-
fn get_usize() -> usize {
46+
const fn get_usize() -> usize {
4747
0
4848
}
4949
fn get_struct() -> Struct {
@@ -117,4 +117,15 @@ fn main() {
117117
'label: {
118118
break 'label
119119
};
120+
let () = const {
121+
[42, 55][get_usize()];
122+
};
123+
}
124+
125+
const _: () = {
126+
[42, 55][get_usize()];
127+
};
128+
129+
const fn foo() {
130+
[42, 55][get_usize()];
120131
}

tests/ui/unnecessary_operation.stderr

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,5 +119,23 @@ LL | | s: String::from("blah"),
119119
LL | | };
120120
| |______^ help: statement can be reduced to: `String::from("blah");`
121121

122-
error: aborting due to 19 previous errors
122+
error: unnecessary operation
123+
--> tests/ui/unnecessary_operation.rs:121:9
124+
|
125+
LL | [42, 55][get_usize()];
126+
| ^^^^^^^^^^^^^^^^^^^^^^ help: statement can be written as: `assert!([42, 55].len() > get_usize());`
127+
128+
error: unnecessary operation
129+
--> tests/ui/unnecessary_operation.rs:126:5
130+
|
131+
LL | [42, 55][get_usize()];
132+
| ^^^^^^^^^^^^^^^^^^^^^^ help: statement can be written as: `assert!([42, 55].len() > get_usize());`
133+
134+
error: unnecessary operation
135+
--> tests/ui/unnecessary_operation.rs:130:5
136+
|
137+
LL | [42, 55][get_usize()];
138+
| ^^^^^^^^^^^^^^^^^^^^^^ help: statement can be written as: `assert!([42, 55].len() > get_usize());`
139+
140+
error: aborting due to 22 previous errors
123141

0 commit comments

Comments
 (0)