Remove unwrap_unchecked in favor of unwrap

Author: Zoxc

src/raw/mod.rs

@@ -155,22 +155,6 @@ fn h2(hash: u64) -> u8 {
     (top7 & 0x7f) as u8 // truncation
 }
 
-/// Returns the contained [`Some`] value, consuming the `this` value,
-/// without checking that the value is not [`None`].
-///
-/// # Safety
-///
-/// Calling this method on [`None`] is *[undefined behavior]*.
-#[inline]
-pub unsafe fn unwrap_unchecked<T>(this: Option<T>) -> T {
-    debug_assert!(this.is_some());
-    match this {
-        Some(val) => val,
-        // SAFETY: the safety contract must be upheld by the caller.
-        None => hint::unreachable_unchecked(),
-    }
-}
-
 /// Probe sequence based on triangular numbers, which is guaranteed (since our
 /// table size is a power of two) to visit every group of elements exactly once.
 ///
@@ -975,8 +959,7 @@ impl<T, A: Allocator + Clone> RawTable<T, A> {
                 let bit = group.match_empty_or_deleted().lowest_set_bit();
 
                 if likely(bit.is_some()) {
-                    let mut index =
-                        (probe_seq.pos + unwrap_unchecked(bit)) & self.table.bucket_mask;
+                    let mut index = (probe_seq.pos + bit.unwrap()) & self.table.bucket_mask;
 
                     // In tables smaller than the group width, trailing control
                     // bytes outside the range of the table are filled with