Skip to content

Commit

Permalink
Move test for 10148 to tests/ui/crashes
Browse files Browse the repository at this point in the history
  • Loading branch information
Alexendoo committed Mar 6, 2023
1 parent dc23e42 commit fa0c3cc
Show file tree
Hide file tree
Showing 5 changed files with 36 additions and 27 deletions.
9 changes: 9 additions & 0 deletions tests/ui/crashes/ice-10148.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
// aux-build:../../auxiliary/proc_macro_with_span.rs

extern crate proc_macro_with_span;

use proc_macro_with_span::with_span;

fn main() {
println!(with_span!(""something ""));
}
12 changes: 12 additions & 0 deletions tests/ui/crashes/ice-10148.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
error: empty string literal in `println!`
--> $DIR/ice-10148.rs:8:5
|
LL | println!(with_span!(""something ""));
| ^^^^^^^^^^^^^^^^^^^^-----------^^^^^
| |
| help: remove the empty string
|
= note: `-D clippy::println-empty-string` implied by `-D warnings`

error: aborting due to previous error

6 changes: 0 additions & 6 deletions tests/ui/format.fixed
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
// run-rustfix
// aux-build: proc_macro_with_span.rs
#![warn(clippy::useless_format)]
#![allow(
unused_tuple_struct_fields,
Expand All @@ -10,8 +9,6 @@
clippy::uninlined_format_args
)]

extern crate proc_macro_with_span;

struct Foo(pub String);

macro_rules! foo {
Expand Down Expand Up @@ -90,7 +87,4 @@ fn main() {
let _ = abc.to_string();
let xx = "xx";
let _ = xx.to_string();

// Issue #10148
println!(proc_macro_with_span::with_span!(""something ""));
}
6 changes: 0 additions & 6 deletions tests/ui/format.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
// run-rustfix
// aux-build: proc_macro_with_span.rs
#![warn(clippy::useless_format)]
#![allow(
unused_tuple_struct_fields,
Expand All @@ -10,8 +9,6 @@
clippy::uninlined_format_args
)]

extern crate proc_macro_with_span;

struct Foo(pub String);

macro_rules! foo {
Expand Down Expand Up @@ -92,7 +89,4 @@ fn main() {
let _ = format!("{abc}");
let xx = "xx";
let _ = format!("{xx}");

// Issue #10148
println!(proc_macro_with_span::with_span!(""something ""));
}
30 changes: 15 additions & 15 deletions tests/ui/format.stderr
Original file line number Diff line number Diff line change
@@ -1,25 +1,25 @@
error: useless use of `format!`
--> $DIR/format.rs:22:5
--> $DIR/format.rs:19:5
|
LL | format!("foo");
| ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"foo".to_string()`
|
= note: `-D clippy::useless-format` implied by `-D warnings`

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

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

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

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

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

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

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

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

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

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

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

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

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

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

0 comments on commit fa0c3cc

Please sign in to comment.