diff --git a/tests/compile-test.rs b/tests/compile-test.rs index 4f17e7ffbc29..583be5a8d47a 100644 --- a/tests/compile-test.rs +++ b/tests/compile-test.rs @@ -345,8 +345,6 @@ const RUSTFIX_COVERAGE_KNOWN_EXCEPTIONS: &[&str] = &[ "cast_size_32bit.rs", "char_lit_as_u8.rs", "cmp_owned/without_suggestion.rs", - "crashes/ice-6250.rs", - "crashes/ice-6251.rs", "dbg_macro.rs", "deref_addrof_double_trigger.rs", "doc/unbalanced_ticks.rs", @@ -388,6 +386,9 @@ fn check_rustfix_coverage() { assert!(RUSTFIX_COVERAGE_KNOWN_EXCEPTIONS.iter().is_sorted_by_key(Path::new)); for rs_path in missing_coverage_contents.lines() { + if Path::new(rs_path).starts_with("tests/ui/crashes") { + continue; + } let filename = Path::new(rs_path).strip_prefix("tests/ui/").unwrap(); assert!( RUSTFIX_COVERAGE_KNOWN_EXCEPTIONS diff --git a/tests/ui/crashes/ice-7169.fixed b/tests/ui/crashes/ice-7169.fixed deleted file mode 100644 index b36fb49ee749..000000000000 --- a/tests/ui/crashes/ice-7169.fixed +++ /dev/null @@ -1,12 +0,0 @@ -// run-rustfix -#![allow(dead_code)] - -#[derive(Default)] -struct A { - a: Vec>, - b: T, -} - -fn main() { - if Ok::<_, ()>(A::::default()).is_ok() {} -} diff --git a/tests/ui/crashes/ice-7169.rs b/tests/ui/crashes/ice-7169.rs index 1769a96bd2f6..82095febc194 100644 --- a/tests/ui/crashes/ice-7169.rs +++ b/tests/ui/crashes/ice-7169.rs @@ -1,6 +1,3 @@ -// run-rustfix -#![allow(dead_code)] - #[derive(Default)] struct A { a: Vec>, diff --git a/tests/ui/crashes/ice-7169.stderr b/tests/ui/crashes/ice-7169.stderr index e7d0671386b6..5a9cd32380a1 100644 --- a/tests/ui/crashes/ice-7169.stderr +++ b/tests/ui/crashes/ice-7169.stderr @@ -1,5 +1,5 @@ error: redundant pattern matching, consider using `is_ok()` - --> $DIR/ice-7169.rs:11:12 + --> $DIR/ice-7169.rs:8:12 | LL | if let Ok(_) = Ok::<_, ()>(A::::default()) {} | -------^^^^^-------------------------------------- help: try this: `if Ok::<_, ()>(A::::default()).is_ok()` diff --git a/tests/ui/crashes/ice-8250.fixed b/tests/ui/crashes/ice-8250.fixed deleted file mode 100644 index 950dc0b4d578..000000000000 --- a/tests/ui/crashes/ice-8250.fixed +++ /dev/null @@ -1,7 +0,0 @@ -// run-rustfix -fn _f(s: &str) -> Option<()> { - let _ = s[1..].split('.').next()?; - Some(()) -} - -fn main() {} diff --git a/tests/ui/crashes/ice-8250.rs b/tests/ui/crashes/ice-8250.rs index d3183d88faee..d9a5ee1162a4 100644 --- a/tests/ui/crashes/ice-8250.rs +++ b/tests/ui/crashes/ice-8250.rs @@ -1,4 +1,3 @@ -// run-rustfix fn _f(s: &str) -> Option<()> { let _ = s[1..].splitn(2, '.').next()?; Some(()) diff --git a/tests/ui/crashes/ice-8250.stderr b/tests/ui/crashes/ice-8250.stderr index 46d5aa7a2fc0..8ed8f3b3a064 100644 --- a/tests/ui/crashes/ice-8250.stderr +++ b/tests/ui/crashes/ice-8250.stderr @@ -1,5 +1,5 @@ error: unnecessary use of `splitn` - --> $DIR/ice-8250.rs:3:13 + --> $DIR/ice-8250.rs:2:13 | LL | let _ = s[1..].splitn(2, '.').next()?; | ^^^^^^^^^^^^^^^^^^^^^ help: try this: `s[1..].split('.')` diff --git a/tests/ui/crashes/ice-8821.fixed b/tests/ui/crashes/ice-8821.fixed deleted file mode 100644 index 046b4522ec1a..000000000000 --- a/tests/ui/crashes/ice-8821.fixed +++ /dev/null @@ -1,9 +0,0 @@ -// run-rustfix -#![warn(clippy::let_unit_value)] - -fn f() {} -static FN: fn() = f; - -fn main() { - FN(); -} diff --git a/tests/ui/crashes/ice-8821.rs b/tests/ui/crashes/ice-8821.rs index 941225e9d1e5..fb87b79aeed8 100644 --- a/tests/ui/crashes/ice-8821.rs +++ b/tests/ui/crashes/ice-8821.rs @@ -1,4 +1,3 @@ -// run-rustfix #![warn(clippy::let_unit_value)] fn f() {} diff --git a/tests/ui/crashes/ice-8821.stderr b/tests/ui/crashes/ice-8821.stderr index aee13f00ca02..486096e0a06d 100644 --- a/tests/ui/crashes/ice-8821.stderr +++ b/tests/ui/crashes/ice-8821.stderr @@ -1,5 +1,5 @@ error: this let-binding has unit value - --> $DIR/ice-8821.rs:8:5 + --> $DIR/ice-8821.rs:7:5 | LL | let _: () = FN(); | ^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `FN();`