f Drop explicit no-std feature

.. by making `hashbrown` a required dependency, just like we did in
`lightning`.

Author: tnull

lightning-liquidity/Cargo.toml

@@ -16,15 +16,14 @@ categories = ["cryptography::cryptocurrencies"]
 [features]
 default = ["std"]
 std = ["lightning/std", "bitcoin/std", "lightning-invoice/std"]
-no-std = ["hashbrown"]
 
 [dependencies]
 lightning = { version = "0.0.124", path = "../lightning", default-features = false }
 lightning-types = { version = "0.1", path = "../lightning-types", default-features = false }
 lightning-invoice = { version = "0.32.0", path = "../lightning-invoice", default-features = false, features = ["serde"] }
 
 bitcoin = { version = "0.32.2", default-features = false, features = ["serde"] }
-hashbrown = { version = "0.8", optional = true }
+hashbrown = { version = "0.8" }
 
 chrono = { version = "0.4", default-features = false, features = ["serde", "alloc"] }
 serde = { version = "1.0", default-features = false, features = ["derive", "alloc"] }

lightning-liquidity/src/lib.rs

@@ -18,22 +18,14 @@
 #![allow(clippy::drop_non_drop)]
 #![cfg_attr(docsrs, feature(doc_auto_cfg))]
 #![cfg_attr(not(feature = "std"), no_std)]
-#[cfg(not(any(feature = "std", feature = "no-std")))]
-compile_error!("at least one of the `std` or `no-std` features must be enabled");
 
 #[macro_use]
 extern crate alloc;
 
 mod prelude {
 	#![allow(unused_imports)]
-	#[cfg(feature = "hashbrown")]
-	extern crate hashbrown;
-
-	#[cfg(feature = "hashbrown")]
-	pub use self::hashbrown::{hash_map, HashMap, HashSet};
 	pub use alloc::{boxed::Box, collections::VecDeque, string::String, vec, vec::Vec};
-	#[cfg(not(feature = "hashbrown"))]
-	pub use std::collections::{hash_map, HashMap, HashSet};
+	pub use hashbrown::{hash_map, HashMap, HashSet};
 
 	pub use alloc::borrow::ToOwned;
 	pub use alloc::string::ToString;

lightning-liquidity/src/manager.rs

@@ -361,7 +361,7 @@ where {
 	/// ```
 	///
 	/// [`PeerManager::process_events`]: lightning::ln::peer_handler::PeerManager::process_events
-	#[cfg(feature = "no-std")]
+	#[cfg(not(feature = "std"))]
 	pub fn set_process_msgs_callback(&self, callback: impl Fn() + 'static) {
 		self.pending_messages.set_process_msgs_callback(callback)
 	}

lightning-liquidity/src/message_queue.rs

@@ -13,7 +13,7 @@ pub struct MessageQueue {
 	queue: Mutex<VecDeque<(PublicKey, LSPSMessage)>>,
 	#[cfg(feature = "std")]
 	process_msgs_callback: RwLock<Option<Box<dyn Fn() + Send + Sync + 'static>>>,
-	#[cfg(feature = "no-std")]
+	#[cfg(not(feature = "std"))]
 	process_msgs_callback: RwLock<Option<Box<dyn Fn() + 'static>>>,
 }
 
@@ -29,7 +29,7 @@ impl MessageQueue {
 		*self.process_msgs_callback.write().unwrap() = Some(Box::new(callback));
 	}
 
-	#[cfg(feature = "no-std")]
+	#[cfg(not(feature = "std"))]
 	pub(crate) fn set_process_msgs_callback(&self, callback: impl Fn() + 'static) {
 		*self.process_msgs_callback.write().unwrap() = Some(Box::new(callback));
 	}