Skip to content

Commit 29211be

Browse files
committed
Add run-rustfix to duration_subsec test
1 parent ea7eb49 commit 29211be

File tree

3 files changed

+36
-5
lines changed

3 files changed

+36
-5
lines changed

tests/ui/duration_subsec.fixed

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
// run-rustfix
2+
#![allow(dead_code)]
3+
#![warn(clippy::duration_subsec)]
4+
5+
use std::time::Duration;
6+
7+
fn main() {
8+
let dur = Duration::new(5, 0);
9+
10+
let bad_millis_1 = dur.subsec_millis();
11+
let bad_millis_2 = dur.subsec_millis();
12+
let good_millis = dur.subsec_millis();
13+
assert_eq!(bad_millis_1, good_millis);
14+
assert_eq!(bad_millis_2, good_millis);
15+
16+
let bad_micros = dur.subsec_micros();
17+
let good_micros = dur.subsec_micros();
18+
assert_eq!(bad_micros, good_micros);
19+
20+
// Handle refs
21+
let _ = (&dur).subsec_micros();
22+
23+
// Handle constants
24+
const NANOS_IN_MICRO: u32 = 1_000;
25+
let _ = dur.subsec_micros();
26+
27+
// Other literals aren't linted
28+
let _ = dur.subsec_nanos() / 699;
29+
}

tests/ui/duration_subsec.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// run-rustfix
2+
#![allow(dead_code)]
13
#![warn(clippy::duration_subsec)]
24

35
use std::time::Duration;

tests/ui/duration_subsec.stderr

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,31 @@
11
error: Calling `subsec_millis()` is more concise than this calculation
2-
--> $DIR/duration_subsec.rs:8:24
2+
--> $DIR/duration_subsec.rs:10:24
33
|
44
LL | let bad_millis_1 = dur.subsec_micros() / 1_000;
55
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `dur.subsec_millis()`
66
|
77
= note: `-D clippy::duration-subsec` implied by `-D warnings`
88

99
error: Calling `subsec_millis()` is more concise than this calculation
10-
--> $DIR/duration_subsec.rs:9:24
10+
--> $DIR/duration_subsec.rs:11:24
1111
|
1212
LL | let bad_millis_2 = dur.subsec_nanos() / 1_000_000;
1313
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `dur.subsec_millis()`
1414

1515
error: Calling `subsec_micros()` is more concise than this calculation
16-
--> $DIR/duration_subsec.rs:14:22
16+
--> $DIR/duration_subsec.rs:16:22
1717
|
1818
LL | let bad_micros = dur.subsec_nanos() / 1_000;
1919
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `dur.subsec_micros()`
2020

2121
error: Calling `subsec_micros()` is more concise than this calculation
22-
--> $DIR/duration_subsec.rs:19:13
22+
--> $DIR/duration_subsec.rs:21:13
2323
|
2424
LL | let _ = (&dur).subsec_nanos() / 1_000;
2525
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `(&dur).subsec_micros()`
2626

2727
error: Calling `subsec_micros()` is more concise than this calculation
28-
--> $DIR/duration_subsec.rs:23:13
28+
--> $DIR/duration_subsec.rs:25:13
2929
|
3030
LL | let _ = dur.subsec_nanos() / NANOS_IN_MICRO;
3131
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `dur.subsec_micros()`

0 commit comments

Comments
 (0)