Review updates

Author: newpavlov

rand_core/src/lib.rs

@@ -200,8 +200,8 @@ pub trait TryRngCore {
     /// Fill `dest` entirely with random data.
     fn try_fill_bytes(&mut self, dst: &mut [u8]) -> Result<(), Self::Error>;
 
-    /// Wrap RNG with the [`PanicOnErr`] wrapper.
-    fn panic_on_err(self) -> UnwrapErr<Self>
+    /// Wrap RNG with the [`UnwrapErr`] wrapper.
+    fn unwrap_err(self) -> UnwrapErr<Self>
     where Self: Sized {
         UnwrapErr(self)
     }
@@ -472,9 +472,6 @@ pub trait SeedableRng: Sized {
     }
 }
 
-// Implement `RngCore` for references to an `RngCore`.
-// Force inlining all functions, so that it is up to the `RngCore`
-// implementation and the optimizer to decide on inlining.
 impl<'a, R: RngCore + ?Sized> RngCore for &'a mut R {
     #[inline(always)]
     fn next_u32(&mut self) -> u32 {
@@ -492,12 +489,8 @@ impl<'a, R: RngCore + ?Sized> RngCore for &'a mut R {
     }
 }
 
-// Implement `TryCryptoRng` for references to a `TryCryptoRng`.
 impl<'a, R: CryptoRng + ?Sized> CryptoRng for &'a mut R {}
 
-// Implement `RngCore` for boxed references to an `RngCore`.
-// Force inlining all functions, so that it is up to the `RngCore`
-// implementation and the optimizer to decide on inlining.
 #[cfg(feature = "alloc")]
 impl<R: RngCore + ?Sized> RngCore for Box<R> {
     #[inline(always)]
@@ -516,7 +509,6 @@ impl<R: RngCore + ?Sized> RngCore for Box<R> {
     }
 }
 
-// Implement `TryCryptoRng` for boxed references to a `TryCryptoRng`.
 #[cfg(feature = "alloc")]
 #[cfg_attr(doc_cfg, doc(cfg(feature = "alloc")))]
 impl<R: CryptoRng + ?Sized> CryptoRng for Box<R> {}

src/rngs/std.rs

@@ -59,11 +59,6 @@ impl SeedableRng for StdRng {
     fn from_seed(seed: Self::Seed) -> Self {
         StdRng(Rng::from_seed(seed))
     }
-
-    #[inline(always)]
-    fn from_rng<R: RngCore>(rng: R) -> Self {
-        Self(Rng::from_rng(rng))
-    }
 }
 
 impl CryptoRng for StdRng {}