From 6ae35be0db84d2bf79ffe0cd63d616a55c036b12 Mon Sep 17 00:00:00 2001
From: "Tobin C. Harding" <me@tobin.cc>
Date: Mon, 9 Dec 2024 13:57:26 +1100
Subject: [PATCH] primitives: Reduce alloc requirements

Recently we reduced the `alloc` requirements in `units` but we did not
propagate these changes up to `primitives`.

Remove a bunch of `alloc` feature gating from `primitives`.
---
 primitives/src/lib.rs      | 7 ++-----
 primitives/src/sequence.rs | 9 ---------
 2 files changed, 2 insertions(+), 14 deletions(-)

diff --git a/primitives/src/lib.rs b/primitives/src/lib.rs
index b8a5204de..032aebcdc 100644
--- a/primitives/src/lib.rs
+++ b/primitives/src/lib.rs
@@ -31,7 +31,6 @@ extern crate std;
 extern crate serde;
 
 pub mod block;
-#[cfg(feature = "alloc")]
 pub mod locktime;
 pub mod merkle_tree;
 pub mod opcodes;
@@ -45,10 +44,8 @@ pub mod transaction;
 pub mod witness;
 
 #[doc(inline)]
-pub use units::amount::{self, Amount, SignedAmount};
-#[doc(inline)]
-#[cfg(feature = "alloc")]
 pub use units::{
+    amount::{self, Amount, SignedAmount},
     block::{BlockHeight, BlockInterval},
     fee_rate::{self, FeeRate},
     weight::{self, Weight},
@@ -60,13 +57,13 @@ pub use self::{
     block::{
         Block, Checked as BlockChecked, Unchecked as BlockUnchecked, Validation as BlockValidation,
     },
-    locktime::{absolute, relative},
     transaction::{Transaction, TxIn, TxOut},
     witness::Witness,
 };
 #[doc(inline)]
 pub use self::{
     block::{BlockHash, Header as BlockHeader, WitnessCommitment},
+    locktime::{absolute, relative},
     merkle_tree::{TxMerkleNode, WitnessMerkleNode},
     pow::CompactTarget,
     sequence::Sequence,
diff --git a/primitives/src/sequence.rs b/primitives/src/sequence.rs
index 5f0233c9c..a0adab268 100644
--- a/primitives/src/sequence.rs
+++ b/primitives/src/sequence.rs
@@ -20,12 +20,9 @@ use core::fmt;
 use arbitrary::{Arbitrary, Unstructured};
 #[cfg(feature = "serde")]
 use serde::{Deserialize, Serialize};
-#[cfg(feature = "alloc")]
 use units::locktime::relative::TimeOverflowError;
-#[cfg(feature = "alloc")]
 use units::parse::{self, PrefixedHexError, UnprefixedHexError};
 
-#[cfg(feature = "alloc")]
 use crate::locktime::relative;
 #[cfg(all(doc, feature = "alloc"))]
 use crate::transaction::Transaction;
@@ -127,14 +124,12 @@ impl Sequence {
     }
 
     /// Constructs a new `Sequence` from a prefixed hex string.
-    #[cfg(feature = "alloc")]
     pub fn from_hex(s: &str) -> Result<Self, PrefixedHexError> {
         let lock_time = parse::hex_u32_prefixed(s)?;
         Ok(Self::from_consensus(lock_time))
     }
 
     /// Constructs a new `Sequence` from an unprefixed hex string.
-    #[cfg(feature = "alloc")]
     pub fn from_unprefixed_hex(s: &str) -> Result<Self, UnprefixedHexError> {
         let lock_time = parse::hex_u32_unprefixed(s)?;
         Ok(Self::from_consensus(lock_time))
@@ -158,7 +153,6 @@ impl Sequence {
     ///
     /// Will return an error if the input cannot be encoded in 16 bits.
     #[inline]
-    #[cfg(feature = "alloc")]
     pub fn from_seconds_floor(seconds: u32) -> Result<Self, TimeOverflowError> {
         if let Ok(interval) = u16::try_from(seconds / 512) {
             Ok(Sequence::from_512_second_intervals(interval))
@@ -172,7 +166,6 @@ impl Sequence {
     ///
     /// Will return an error if the input cannot be encoded in 16 bits.
     #[inline]
-    #[cfg(feature = "alloc")]
     pub fn from_seconds_ceil(seconds: u32) -> Result<Self, TimeOverflowError> {
         if let Ok(interval) = u16::try_from((seconds + 511) / 512) {
             Ok(Sequence::from_512_second_intervals(interval))
@@ -191,7 +184,6 @@ impl Sequence {
 
     /// Constructs a new [`relative::LockTime`] from this [`Sequence`] number.
     #[inline]
-    #[cfg(feature = "alloc")]
     pub fn to_relative_lock_time(&self) -> Option<relative::LockTime> {
         use crate::locktime::relative::{Height, LockTime, Time};
 
@@ -211,7 +203,6 @@ impl Sequence {
     /// Returns the low 16 bits from sequence number.
     ///
     /// BIP-68 only uses the low 16 bits for relative lock value.
-    #[cfg(feature = "alloc")]
     fn low_u16(&self) -> u16 { self.0 as u16 }
 }