Remove unused cfg attributes (#361)

* Remove unused cfg attributes

* code quality: remove unused cfg attributes and fix warning

crates/floresta-chain/src/lib.rs

@@ -10,7 +10,7 @@
 //! All data is stored in a `ChainStore` implementation, which is generic over the
 //! underlying database. See the ChainStore trait for more information. For a
 //! ready-to-use implementation, see the [KvChainStore] struct.
-#![cfg_attr(any(feature = "no-std", not(test)), no_std)]
+#![cfg_attr(not(test), no_std)]
 
 pub mod pruned_utreexo;
 pub(crate) use floresta_common::prelude;

crates/floresta-chain/src/pruned_utreexo/chain_state.rs

@@ -1,6 +1,4 @@
 extern crate alloc;
-#[cfg(not(feature = "no-std"))]
-extern crate std;
 
 use alloc::borrow::ToOwned;
 use alloc::fmt::format;

crates/floresta-chain/src/pruned_utreexo/error.rs

@@ -3,18 +3,13 @@ use core::fmt::Debug;
 use bitcoin::blockdata::script;
 use bitcoin::OutPoint;
 use bitcoin::Txid;
-#[cfg(feature = "cli-blockchain")]
-use btcd_rpc::error::UtreexodError;
 use floresta_common::impl_error_from;
 
 use crate::prelude::*;
 pub trait DatabaseError: Debug + Send + Sync + 'static {}
 #[derive(Debug)]
 pub enum BlockchainError {
     BlockNotPresent,
-    #[cfg(feature = "cli-blockchain")]
-    #[error("Json-Rpc error")]
-    JsonRpcError(#[from] UtreexodError),
     Parsing(String),
     BlockValidation(BlockValidationErrors),
     TransactionError(TransactionError),

crates/floresta-electrum/src/error.rs

@@ -1,12 +1,7 @@
-#[cfg(feature = "cli-blockchain")]
-use btcd_rpc::error::UtreexodError;
 use thiserror::Error;
 
 #[derive(Error, Debug)]
 pub enum Error {
-    #[error("Utreexod error")]
-    #[cfg(feature = "cli-blockchain")]
-    BackendError(#[from] UtreexodError),
     #[error("Invalid params passed in")]
     InvalidParams,
     #[error("Invalid json string {0}")]

crates/floresta-watch-only/src/lib.rs

@@ -1,6 +1,4 @@
 // SPDX-License-Identifier: MIT
-
-#![cfg_attr(feature = "no-std", no_std)]
 use core::cmp::Ordering;
 use core::fmt::Debug;
 

florestad/src/error.rs

@@ -1,14 +1,10 @@
 use bitcoin::consensus::encode;
-#[cfg(feature = "cli-blockchain")]
-use btcd_rpc::error::UtreexodError;
 use floresta_chain::BlockValidationErrors;
 use floresta_chain::BlockchainError;
 
 use crate::slip132;
 #[derive(Debug)]
 pub enum Error {
-    #[cfg(feature = "cli-blockchain")]
-    UtreexodError(UtreexodError),
     Encode(encode::Error),
     Db(kv::Error),
     ParseNum(std::num::ParseIntError),
@@ -28,8 +24,6 @@ impl std::fmt::Display for Error {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {
             Error::Encode(err) => write!(f, "Encode error: {err}"),
-            #[cfg(feature = "cli-blockchain")]
-            Error::UtreexodError(_) => write!(f, "UtreexodError"),
             Error::Db(err) => write!(f, "Database error {err}"),
             Error::ParseNum(err) => write!(f, "int parse error: {err}"),
             Error::Rustreexo(err) => write!(f, "Rustreexo error: {err}"),
@@ -56,9 +50,7 @@ macro_rules! impl_from_error {
         }
     };
 }
-// impl_from_error!(Parsing, bitcoin::hashes::hex::Error);
-#[cfg(feature = "cli-blockchain")]
-impl_from_error!(UtreexodError, UtreexodError);
+
 impl_from_error!(Encode, encode::Error);
 impl_from_error!(Db, kv::Error);
 impl_from_error!(ParseNum, std::num::ParseIntError);

florestad/src/slip132.rs

@@ -152,7 +152,6 @@ pub struct DefaultResolver;
     derive(Serialize, Deserialize),
     serde(crate = "serde_crate")
 )]
-#[cfg_attr(feature = "strict_encoding", derive(StrictEncode, StrictDecode))]
 #[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Debug)]
 #[non_exhaustive]
 pub enum KeyApplication {