about summary refs log tree commit diff
diff options
context:
space:
mode:
authorSabrinaJewson <sejewson@gmail.com>2022-04-01 06:32:22 +0100
committerSabrinaJewson <sejewson@gmail.com>2022-04-01 06:32:22 +0100
commit11045f94e267805e90662577ee7486cba5dbcf18 (patch)
tree91f038d14cb015dbd44f9f217043841e5e732816
parent7a80c23f838b832e651a717767c41a2536f3ae42 (diff)
downloadrust-11045f94e267805e90662577ee7486cba5dbcf18.tar.gz
rust-11045f94e267805e90662577ee7486cba5dbcf18.zip
Don't unnecessarily suggest unsafe block
-rw-r--r--clippy_lints/src/transmute/transmute_ref_to_ref.rs2
-rw-r--r--tests/ui/transmute.stderr2
2 files changed, 2 insertions, 2 deletions
diff --git a/clippy_lints/src/transmute/transmute_ref_to_ref.rs b/clippy_lints/src/transmute/transmute_ref_to_ref.rs
index 9c893cacf4f..fe3e88613f4 100644
--- a/clippy_lints/src/transmute/transmute_ref_to_ref.rs
+++ b/clippy_lints/src/transmute/transmute_ref_to_ref.rs
@@ -41,7 +41,7 @@ pub(super) fn check<'tcx>(
                     &format!("transmute from a `{}` to a `{}`", from_ty, to_ty),
                     "consider using",
                     if const_context {
-                        format!("unsafe {{ std::str::from_utf8_unchecked{postfix}({snippet}) }}")
+                        format!("std::str::from_utf8_unchecked{postfix}({snippet})")
                     } else {
                         format!("std::str::from_utf8{postfix}({snippet}).unwrap()")
                     },
diff --git a/tests/ui/transmute.stderr b/tests/ui/transmute.stderr
index 1213d192ae9..72a386e8fa6 100644
--- a/tests/ui/transmute.stderr
+++ b/tests/ui/transmute.stderr
@@ -244,7 +244,7 @@ error: transmute from a `&[u8]` to a `&str`
   --> $DIR/transmute.rs:142:30
    |
 LL |     const _: &str = unsafe { std::mem::transmute(B) };
-   |                              ^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `unsafe { std::str::from_utf8_unchecked(B) }`
+   |                              ^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `std::str::from_utf8_unchecked(B)`
 
 error: aborting due to 39 previous errors