f sp and condense possiblyrandom

Author: TheBlueMatt

possiblyrandom/Cargo.toml

@@ -5,7 +5,7 @@ authors = ["Matt Corallo"]
 license = "MIT OR Apache-2.0"
 repository = "https://github.com/lightningdevkit/rust-lightning/"
 description = """
-A crate which wraps getrandom and always compiles, returning 0s when no randomness is available.
+A crate that wraps getrandom and always compiles, returning 0s when no randomness is available.
 """
 edition = "2021"
 

possiblyrandom/src/lib.rs

@@ -7,7 +7,7 @@
 // You may not use this file except in accordance with one or both of these
 // licenses.
 
-//! [`getrandom`] provides access to OS randomness, but will fail to compile on platforms which do
+//! [`getrandom`] provides access to OS randomness, but will fail to compile on platforms that do
 //! not support fetching OS randomness. This is exactly what you want when you're doing
 //! cryptographic operations, but when you're just opportunistically randomizing, we're fine with
 //! compiling and simply disabling randomization.
@@ -24,17 +24,12 @@
 extern crate getrandom;
 
 /// Possibly fills `dest` with random data. May fill it with zeros.
-#[cfg(feature = "getrandom")]
 #[inline]
 pub fn getpossiblyrandom(dest: &mut [u8]) {
+	#[cfg(feature = "getrandom")]
 	if getrandom::getrandom(dest).is_err() {
 		dest.fill(0);
 	}
-}
-
-/// Possibly fills `dest` with random data. May fill it with zeros.
-#[cfg(not(feature = "getrandom"))]
-#[inline]
-pub fn getpossiblyrandom(dest: &mut [u8]) {
+	#[cfg(not(feature = "getrandom"))]
 	dest.fill(0);
 }