Skip to content

Commit

Permalink
Auto merge of #75157 - rodrimati1992:patch-1, r=oli-obk
Browse files Browse the repository at this point in the history
Constified str::from_utf8_unchecked

This would be useful for const code to use an array to construct a string using guaranteed utf8 inputs, and then create a `&str` from it.
  • Loading branch information
bors committed Aug 14, 2020
2 parents 7996182 + 1837708 commit 55b9adf
Showing 1 changed file with 9 additions and 13 deletions.
22 changes: 9 additions & 13 deletions library/core/src/str/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -414,12 +414,13 @@ pub fn from_utf8_mut(v: &mut [u8]) -> Result<&mut str, Utf8Error> {
/// ```
#[inline]
#[stable(feature = "rust1", since = "1.0.0")]
pub unsafe fn from_utf8_unchecked(v: &[u8]) -> &str {
// SAFETY: the caller must guarantee that the bytes `v`
// are valid UTF-8, thus the cast to `*const str` is safe.
// Also, the pointer dereference is safe because that pointer
// comes from a reference which is guaranteed to be valid for reads.
unsafe { &*(v as *const [u8] as *const str) }
#[rustc_const_unstable(feature = "const_str_from_utf8_unchecked", issue = "75196")]
#[allow(unused_attributes)]
#[allow_internal_unstable(const_fn_transmute)]
pub const unsafe fn from_utf8_unchecked(v: &[u8]) -> &str {
// SAFETY: the caller must guarantee that the bytes `v` are valid UTF-8.
// Also relies on `&str` and `&[u8]` having the same layout.
unsafe { mem::transmute(v) }
}

/// Converts a slice of bytes to a string slice without checking
Expand Down Expand Up @@ -2357,15 +2358,10 @@ impl str {
#[rustc_const_stable(feature = "str_as_bytes", since = "1.32.0")]
#[inline(always)]
#[allow(unused_attributes)]
#[allow_internal_unstable(const_fn_union)]
#[allow_internal_unstable(const_fn_transmute)]
pub const fn as_bytes(&self) -> &[u8] {
#[repr(C)]
union Slices<'a> {
str: &'a str,
slice: &'a [u8],
}
// SAFETY: const sound because we transmute two types with the same layout
unsafe { Slices { str: self }.slice }
unsafe { mem::transmute(self) }
}

/// Converts a mutable string slice to a mutable byte slice.
Expand Down

0 comments on commit 55b9adf

Please sign in to comment.