Import alloc types directly in lightning-liquidity

While using a `prelude` can be helpful when we import `*` from it,
importing individual types through a `prelude` is somewhat
unnecessary indirection. Further, importing `alloc` types like
`String` and `Vec` through a `prelude` confuse the bindings
generator.

Thus, here we drop the `alloc` imports from the
`lightning-liquidity` `prelude` and replace it with direct imports.

We leave the `lightning-liquidity::prelude::hash_tables` `pub(use)`
in place as the sym-linked `debug_sync` relies on it.

Author: TheBlueMatt

lightning-liquidity/src/events.rs

@@ -18,9 +18,11 @@
 use crate::lsps0;
 use crate::lsps1;
 use crate::lsps2;
-use crate::prelude::{Vec, VecDeque};
 use crate::sync::{Arc, Mutex};
 
+use alloc::collections::VecDeque;
+use alloc::vec::Vec;
+
 use core::future::Future;
 use core::task::{Poll, Waker};
 

lightning-liquidity/src/lib.rs

@@ -52,12 +52,6 @@
 extern crate alloc;
 
 mod prelude {
-	#![allow(unused_imports)]
-	pub use alloc::{boxed::Box, collections::VecDeque, string::String, vec, vec::Vec};
-
-	pub use alloc::borrow::ToOwned;
-	pub use alloc::string::ToString;
-
 	pub(crate) use lightning::util::hash_tables::*;
 }
 

lightning-liquidity/src/lsps0/event.rs

@@ -9,7 +9,8 @@
 
 //! Contains bLIP-50 / LSPS0 event types.
 
-use crate::prelude::Vec;
+use alloc::vec::Vec;
+
 use bitcoin::secp256k1::PublicKey;
 
 /// An event which an bLIP-50 / LSPS0 client may want to take some action in response to.

lightning-liquidity/src/lsps0/msgs.rs

@@ -1,12 +1,12 @@
 //! Message, request, and other primitive types used to implement LSPS0.
 
+use alloc::vec::Vec;
+use core::convert::TryFrom;
+
 use crate::lsps0::ser::{LSPSMessage, LSPSRequestId, LSPSResponseError};
-use crate::prelude::Vec;
 
 use serde::{Deserialize, Serialize};
 
-use core::convert::TryFrom;
-
 pub(crate) const LSPS0_LISTPROTOCOLS_METHOD_NAME: &str = "lsps0.list_protocols";
 
 /// A `list_protocols` request.
@@ -99,7 +99,8 @@ mod tests {
 
 	use super::*;
 	use crate::lsps0::ser::LSPSMethod;
-	use crate::prelude::ToString;
+
+	use alloc::string::ToString;
 
 	#[test]
 	fn deserializes_request() {

lightning-liquidity/src/lsps0/ser.rs

@@ -4,6 +4,11 @@
 //! specification](https://github.com/lightning/blips/blob/master/blip-0050.md) for more
 //! information.
 
+use alloc::string::String;
+
+use core::fmt::{self, Display};
+use core::str::FromStr;
+
 use crate::lsps0::msgs::{
 	LSPS0ListProtocolsRequest, LSPS0Message, LSPS0Request, LSPS0Response,
 	LSPS0_LISTPROTOCOLS_METHOD_NAME,
@@ -16,17 +21,14 @@ use crate::lsps1::msgs::{
 use crate::lsps2::msgs::{
 	LSPS2Message, LSPS2Request, LSPS2Response, LSPS2_BUY_METHOD_NAME, LSPS2_GET_INFO_METHOD_NAME,
 };
-use crate::prelude::{HashMap, String};
+use crate::prelude::HashMap;
 
 use lightning::ln::msgs::{DecodeError, LightningError};
 use lightning::ln::wire;
 use lightning::util::ser::{LengthLimitedRead, LengthReadable, WithoutLength};
 
 use bitcoin::secp256k1::PublicKey;
 
-use core::fmt::{self, Display};
-use core::str::FromStr;
-
 #[cfg(feature = "std")]
 use std::time::{SystemTime, UNIX_EPOCH};
 
@@ -625,7 +627,7 @@ impl<'de, 'a> Visitor<'de> for LSPSMessageVisitor<'a> {
 }
 
 pub(crate) mod string_amount {
-	use crate::prelude::{String, ToString};
+	use alloc::string::{String, ToString};
 	use core::str::FromStr;
 	use serde::de::Unexpected;
 	use serde::{Deserialize, Deserializer, Serializer};
@@ -650,7 +652,7 @@ pub(crate) mod string_amount {
 }
 
 pub(crate) mod string_amount_option {
-	use crate::prelude::{String, ToString};
+	use alloc::string::{String, ToString};
 	use core::str::FromStr;
 	use serde::de::Unexpected;
 	use serde::{Deserialize, Deserializer, Serialize, Serializer};
@@ -679,7 +681,7 @@ pub(crate) mod string_amount_option {
 }
 
 pub(crate) mod string_offer {
-	use crate::prelude::{String, ToString};
+	use alloc::string::{String, ToString};
 	use core::str::FromStr;
 	use lightning::offers::offer::Offer;
 	use serde::de::Unexpected;
@@ -705,7 +707,7 @@ pub(crate) mod string_offer {
 }
 
 pub(crate) mod unchecked_address {
-	use crate::prelude::{String, ToString};
+	use alloc::string::{String, ToString};
 	use bitcoin::Address;
 	use core::str::FromStr;
 	use serde::de::Unexpected;
@@ -732,7 +734,7 @@ pub(crate) mod unchecked_address {
 }
 
 pub(crate) mod unchecked_address_option {
-	use crate::prelude::{String, ToString};
+	use alloc::string::{String, ToString};
 	use bitcoin::Address;
 	use core::str::FromStr;
 	use serde::de::Unexpected;

lightning-liquidity/src/lsps0/service.rs

@@ -13,10 +13,11 @@
 //! specifcation](https://github.com/lightning/blips/blob/master/blip-0050.md) for more
 //! information.
 
+use alloc::vec::Vec;
+
 use crate::lsps0::msgs::{LSPS0ListProtocolsResponse, LSPS0Message, LSPS0Request, LSPS0Response};
 use crate::lsps0::ser::{LSPSProtocolMessageHandler, LSPSRequestId};
 use crate::message_queue::MessageQueue;
-use crate::prelude::Vec;
 use crate::sync::Arc;
 
 use lightning::ln::msgs::{ErrorAction, LightningError};

lightning-liquidity/src/lsps1/msgs.rs

@@ -1,21 +1,21 @@
 //! Message, request, and other primitive types used to implement bLIP-51 / LSPS1.
 
+use alloc::string::String;
+
+use core::convert::TryFrom;
+
 use crate::lsps0::ser::{
 	string_amount, string_offer, u32_fee_rate, unchecked_address, unchecked_address_option,
 	LSPSDateTime, LSPSMessage, LSPSRequestId, LSPSResponseError,
 };
 
-use crate::prelude::String;
-
 use bitcoin::{Address, FeeRate, OutPoint};
 
 use lightning::offers::offer::Offer;
 use lightning_invoice::Bolt11Invoice;
 
 use serde::{Deserialize, Serialize};
 
-use core::convert::TryFrom;
-
 pub(crate) const LSPS1_GET_INFO_METHOD_NAME: &str = "lsps1.get_info";
 pub(crate) const LSPS1_CREATE_ORDER_METHOD_NAME: &str = "lsps1.create_order";
 pub(crate) const LSPS1_GET_ORDER_METHOD_NAME: &str = "lsps1.get_order";

lightning-liquidity/src/lsps1/service.rs

@@ -9,6 +9,10 @@
 
 //! Contains the main bLIP-51 / LSPS1 server object, [`LSPS1ServiceHandler`].
 
+use alloc::string::String;
+
+use core::ops::Deref;
+
 use super::event::LSPS1ServiceEvent;
 use super::msgs::{
 	LSPS1ChannelInfo, LSPS1CreateOrderRequest, LSPS1CreateOrderResponse, LSPS1GetInfoResponse,
@@ -22,7 +26,7 @@ use crate::events::EventQueue;
 use crate::lsps0::ser::{
 	LSPSDateTime, LSPSProtocolMessageHandler, LSPSRequestId, LSPSResponseError,
 };
-use crate::prelude::{new_hash_map, HashMap, String};
+use crate::prelude::{new_hash_map, HashMap};
 use crate::sync::{Arc, Mutex, RwLock};
 use crate::utils;
 
@@ -36,7 +40,6 @@ use lightning::util::logger::Level;
 use bitcoin::secp256k1::PublicKey;
 
 use chrono::Utc;
-use core::ops::Deref;
 
 /// Server-side configuration options for bLIP-51 / LSPS1 channel requests.
 #[derive(Clone, Debug)]

lightning-liquidity/src/lsps2/client.rs

@@ -8,11 +8,16 @@
 
 //! Contains the main bLIP-52 / LSPS2 client object, [`LSPS2ClientHandler`].
 
+use alloc::string::{String, ToString};
+
+use core::default::Default;
+use core::ops::Deref;
+
 use crate::events::EventQueue;
 use crate::lsps0::ser::{LSPSProtocolMessageHandler, LSPSRequestId, LSPSResponseError};
 use crate::lsps2::event::LSPS2ClientEvent;
 use crate::message_queue::MessageQueue;
-use crate::prelude::{new_hash_map, new_hash_set, HashMap, HashSet, String, ToString};
+use crate::prelude::{new_hash_map, new_hash_set, HashMap, HashSet};
 use crate::sync::{Arc, Mutex, RwLock};
 
 use lightning::ln::msgs::{ErrorAction, LightningError};
@@ -22,9 +27,6 @@ use lightning::util::logger::Level;
 
 use bitcoin::secp256k1::PublicKey;
 
-use core::default::Default;
-use core::ops::Deref;
-
 use crate::lsps2::msgs::{
 	LSPS2BuyRequest, LSPS2BuyResponse, LSPS2GetInfoRequest, LSPS2GetInfoResponse, LSPS2Message,
 	LSPS2OpeningFeeParams, LSPS2Request, LSPS2Response,

lightning-liquidity/src/lsps2/event.rs

@@ -11,7 +11,8 @@
 
 use super::msgs::LSPS2OpeningFeeParams;
 use crate::lsps0::ser::LSPSRequestId;
-use crate::prelude::{String, Vec};
+use alloc::string::String;
+use alloc::vec::Vec;
 
 use bitcoin::secp256k1::PublicKey;