Fix rustc warnings

Signed-off-by: Denis Varlakov <[email protected]>
LFDT-Lockness · Oct 2, 2024 · 0eefdda · 0eefdda
1 parent 95afc83
commit 0eefdda
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 0 deletions.
diff --git a/givre/src/lib.rs b/givre/src/lib.rs
@@ -204,6 +204,9 @@ pub mod key_share {
 }
 
 mod error {
+    // It's currently only needed in full signing, but better keep it always in the code
+    #![cfg_attr(not(feature = "full-signing"), allow(dead_code, unused_imports))]
+
     #[cfg(feature = "std")]
     pub use std::error::Error as StdError;
 

diff --git a/givre/src/signing/aggregate.rs b/givre/src/signing/aggregate.rs
@@ -183,6 +183,7 @@ impl<'a, C: Ciphersuite> AggregateOptions<'a, C> {
     ///
     /// CAUTION: additive shift MUST BE derived from the extended public key obtained from
     /// the key share which is used for signing by calling [`utils::derive_additive_shift`].
+    #[cfg(feature = "hd-wallet")]
     pub(crate) fn dangerous_set_hd_additive_shift(
         mut self,
         hd_additive_shift: Scalar<C::Curve>,

diff --git a/givre/src/signing/full_signing.rs b/givre/src/signing/full_signing.rs
@@ -309,9 +309,11 @@ enum SigningOutput<C: Ciphersuite> {
 
 /// Interactive Signing error
 #[derive(Debug)]
+#[cfg_attr(not(feature = "std"), allow(dead_code))]
 pub struct FullSigningError(Reason);
 
 #[derive(Debug)]
+#[cfg_attr(not(feature = "std"), allow(dead_code))]
 enum Reason {
     NonTaprootCiphersuite,
     NOverflowsU16,
@@ -324,6 +326,7 @@ enum Reason {
 }
 
 #[derive(Debug)]
+#[cfg_attr(not(feature = "std"), allow(dead_code))]
 enum IoError {
     Send(Box<dyn crate::error::StdError + Send + Sync>),
     Recv(Box<dyn crate::error::StdError + Send + Sync>),
@@ -339,6 +342,7 @@ impl IoError {
 }
 
 #[derive(Debug)]
+#[cfg_attr(not(feature = "std"), allow(dead_code))]
 enum Bug {
     UnexpectedOutput,
     AdditiveShiftWithoutHdFeature,

diff --git a/givre/src/signing/round2.rs b/givre/src/signing/round2.rs
@@ -135,6 +135,7 @@ impl<'a, C: Ciphersuite> SigningOptions<'a, C> {
     ///
     /// CAUTION: additive shift MUST BE derived from the extended public key obtained from
     /// the key share which is used for signing by calling [`utils::derive_additive_shift`].
+    #[cfg(feature = "hd-wallet")]
     pub(crate) fn dangerous_set_hd_additive_shift(
         mut self,
         hd_additive_shift: Scalar<C::Curve>,
@@ -450,6 +451,7 @@ fn normalize_key_share<C: Ciphersuite>(
 pub struct SigningError(Reason);
 
 #[derive(Debug)]
+#[cfg_attr(not(feature = "std"), allow(dead_code))]
 enum Reason {
     #[cfg(feature = "taproot")]
     MissingTaprootMerkleRoot,