Use `unwrap` instead of `unwrap_unchecked`

Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>

1 files changed, 7 insertions, 3 deletions

diff --git a/library/std/src/io/error.rs b/library/std/src/io/error.rs
index b1931a52290..4c6fd55ad13 100644
--- a/library/std/src/io/error.rs
+++ b/library/std/src/io/error.rs
@@ -844,9 +844,13 @@ impl Error {
             ErrorData::Custom(b) if b.error.is::<E>() => {
                 let res = (*b).error.downcast::<E>();
 
-                // Safety: b.error.is::<E>() returns true,
-                // which means that res must be Ok(e).
-                Ok(unsafe { res.unwrap_unchecked() })
+                // downcast is a really trivial and is marked as inline, so
+                // it's likely be inlined here.
+                //
+                // And the compiler should be able to eliminate the branch
+                // that produces `Err` here since b.error.is::<E>()
+                // returns true.
+                Ok(res.unwrap())
             }
             repr_data => Err(Self { repr: Repr::new(repr_data) }),
         }