Skip to content

Commit fa0c3cc

Browse files
committed
Move test for 10148 to tests/ui/crashes
1 parent dc23e42 commit fa0c3cc

File tree

5 files changed

+36
-27
lines changed

5 files changed

+36
-27
lines changed

tests/ui/crashes/ice-10148.rs

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
// aux-build:../../auxiliary/proc_macro_with_span.rs
2+
3+
extern crate proc_macro_with_span;
4+
5+
use proc_macro_with_span::with_span;
6+
7+
fn main() {
8+
println!(with_span!(""something ""));
9+
}

tests/ui/crashes/ice-10148.stderr

+12
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
error: empty string literal in `println!`
2+
--> $DIR/ice-10148.rs:8:5
3+
|
4+
LL | println!(with_span!(""something ""));
5+
| ^^^^^^^^^^^^^^^^^^^^-----------^^^^^
6+
| |
7+
| help: remove the empty string
8+
|
9+
= note: `-D clippy::println-empty-string` implied by `-D warnings`
10+
11+
error: aborting due to previous error
12+

tests/ui/format.fixed

-6
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
// run-rustfix
2-
// aux-build: proc_macro_with_span.rs
32
#![warn(clippy::useless_format)]
43
#![allow(
54
unused_tuple_struct_fields,
@@ -10,8 +9,6 @@
109
clippy::uninlined_format_args
1110
)]
1211

13-
extern crate proc_macro_with_span;
14-
1512
struct Foo(pub String);
1613

1714
macro_rules! foo {
@@ -90,7 +87,4 @@ fn main() {
9087
let _ = abc.to_string();
9188
let xx = "xx";
9289
let _ = xx.to_string();
93-
94-
// Issue #10148
95-
println!(proc_macro_with_span::with_span!(""something ""));
9690
}

tests/ui/format.rs

-6
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
// run-rustfix
2-
// aux-build: proc_macro_with_span.rs
32
#![warn(clippy::useless_format)]
43
#![allow(
54
unused_tuple_struct_fields,
@@ -10,8 +9,6 @@
109
clippy::uninlined_format_args
1110
)]
1211

13-
extern crate proc_macro_with_span;
14-
1512
struct Foo(pub String);
1613

1714
macro_rules! foo {
@@ -92,7 +89,4 @@ fn main() {
9289
let _ = format!("{abc}");
9390
let xx = "xx";
9491
let _ = format!("{xx}");
95-
96-
// Issue #10148
97-
println!(proc_macro_with_span::with_span!(""something ""));
9892
}

tests/ui/format.stderr

+15-15
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,25 @@
11
error: useless use of `format!`
2-
--> $DIR/format.rs:22:5
2+
--> $DIR/format.rs:19:5
33
|
44
LL | format!("foo");
55
| ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"foo".to_string()`
66
|
77
= note: `-D clippy::useless-format` implied by `-D warnings`
88

99
error: useless use of `format!`
10-
--> $DIR/format.rs:23:5
10+
--> $DIR/format.rs:20:5
1111
|
1212
LL | format!("{{}}");
1313
| ^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"{}".to_string()`
1414

1515
error: useless use of `format!`
16-
--> $DIR/format.rs:24:5
16+
--> $DIR/format.rs:21:5
1717
|
1818
LL | format!("{{}} abc {{}}");
1919
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"{} abc {}".to_string()`
2020

2121
error: useless use of `format!`
22-
--> $DIR/format.rs:25:5
22+
--> $DIR/format.rs:22:5
2323
|
2424
LL | / format!(
2525
LL | | r##"foo {{}}
@@ -34,67 +34,67 @@ LL ~ " bar"##.to_string();
3434
|
3535

3636
error: useless use of `format!`
37-
--> $DIR/format.rs:30:13
37+
--> $DIR/format.rs:27:13
3838
|
3939
LL | let _ = format!("");
4040
| ^^^^^^^^^^^ help: consider using `String::new()`: `String::new()`
4141

4242
error: useless use of `format!`
43-
--> $DIR/format.rs:32:5
43+
--> $DIR/format.rs:29:5
4444
|
4545
LL | format!("{}", "foo");
4646
| ^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"foo".to_string()`
4747

4848
error: useless use of `format!`
49-
--> $DIR/format.rs:40:5
49+
--> $DIR/format.rs:37:5
5050
|
5151
LL | format!("{}", arg);
5252
| ^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `arg.to_string()`
5353

5454
error: useless use of `format!`
55-
--> $DIR/format.rs:70:5
55+
--> $DIR/format.rs:67:5
5656
|
5757
LL | format!("{}", 42.to_string());
5858
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `42.to_string()`
5959

6060
error: useless use of `format!`
61-
--> $DIR/format.rs:72:5
61+
--> $DIR/format.rs:69:5
6262
|
6363
LL | format!("{}", x.display().to_string());
6464
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `x.display().to_string()`
6565

6666
error: useless use of `format!`
67-
--> $DIR/format.rs:76:18
67+
--> $DIR/format.rs:73:18
6868
|
6969
LL | let _ = Some(format!("{}", a + "bar"));
7070
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `a + "bar"`
7171

7272
error: useless use of `format!`
73-
--> $DIR/format.rs:80:22
73+
--> $DIR/format.rs:77:22
7474
|
7575
LL | let _s: String = format!("{}", &*v.join("/n"));
7676
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `(&*v.join("/n")).to_string()`
7777

7878
error: useless use of `format!`
79-
--> $DIR/format.rs:86:13
79+
--> $DIR/format.rs:83:13
8080
|
8181
LL | let _ = format!("{x}");
8282
| ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `x.to_string()`
8383

8484
error: useless use of `format!`
85-
--> $DIR/format.rs:88:13
85+
--> $DIR/format.rs:85:13
8686
|
8787
LL | let _ = format!("{y}", y = x);
8888
| ^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `x.to_string()`
8989

9090
error: useless use of `format!`
91-
--> $DIR/format.rs:92:13
91+
--> $DIR/format.rs:89:13
9292
|
9393
LL | let _ = format!("{abc}");
9494
| ^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `abc.to_string()`
9595

9696
error: useless use of `format!`
97-
--> $DIR/format.rs:94:13
97+
--> $DIR/format.rs:91:13
9898
|
9999
LL | let _ = format!("{xx}");
100100
| ^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `xx.to_string()`

0 commit comments

Comments
 (0)