chore(hlapi): fix clippy lints

zama-ai · Jun 26, 2024 · 3573f0e · 3573f0e
1 parent ebfa153
commit 3573f0e
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 10 deletions.
diff --git a/tfhe/src/high_level_api/booleans/inner.rs b/tfhe/src/high_level_api/booleans/inner.rs
@@ -1,7 +1,8 @@
 use crate::backward_compatibility::booleans::InnerBooleanVersionedOwned;
 use crate::high_level_api::details::MaybeCloned;
+use crate::high_level_api::global_state;
 #[cfg(feature = "gpu")]
-use crate::high_level_api::global_state::{self, with_thread_local_cuda_streams};
+use crate::high_level_api::global_state::with_thread_local_cuda_streams;
 use crate::integer::BooleanBlock;
 use crate::Device;
 use serde::{Deserializer, Serializer};
@@ -219,9 +220,7 @@ impl InnerBoolean {
     }
 
     #[inline]
-    #[allow(clippy::unused_self, clippy::needless_pass_by_ref_mut)]
     pub(crate) fn move_to_device_of_server_key_if_set(&mut self) {
-        #[cfg(feature = "gpu")]
         if let Some(device) = global_state::device_of_internal_keys() {
             self.move_to_device(device);
         }

diff --git a/tfhe/src/high_level_api/global_state.rs b/tfhe/src/high_level_api/global_state.rs
@@ -109,7 +109,6 @@ where
     })
 }
 
-#[cfg(feature = "gpu")]
 #[inline]
 pub(in crate::high_level_api) fn device_of_internal_keys() -> Option<crate::Device> {
     // Should use `with_borrow` when its stabilized

diff --git a/tfhe/src/high_level_api/integers/signed/inner.rs b/tfhe/src/high_level_api/integers/signed/inner.rs
@@ -1,7 +1,8 @@
 use crate::backward_compatibility::integers::SignedRadixCiphertextVersionedOwned;
 use crate::high_level_api::details::MaybeCloned;
+use crate::high_level_api::global_state;
 #[cfg(feature = "gpu")]
-use crate::high_level_api::global_state::{self, with_thread_local_cuda_streams};
+use crate::high_level_api::global_state::with_thread_local_cuda_streams;
 #[cfg(feature = "gpu")]
 use crate::integer::gpu::ciphertext::CudaIntegerRadixCiphertext;
 #[cfg(feature = "gpu")]
@@ -208,9 +209,7 @@ impl RadixCiphertext {
     }
 
     #[inline]
-    #[allow(clippy::unused_self, clippy::needless_pass_by_ref_mut)]
     pub(crate) fn move_to_device_of_server_key_if_set(&mut self) {
-        #[cfg(feature = "gpu")]
         if let Some(device) = global_state::device_of_internal_keys() {
             self.move_to_device(device);
         }

diff --git a/tfhe/src/high_level_api/integers/unsigned/inner.rs b/tfhe/src/high_level_api/integers/unsigned/inner.rs
@@ -1,7 +1,8 @@
 use crate::backward_compatibility::integers::UnsignedRadixCiphertextVersionedOwned;
 use crate::high_level_api::details::MaybeCloned;
+use crate::high_level_api::global_state;
 #[cfg(feature = "gpu")]
-use crate::high_level_api::global_state::{self, with_thread_local_cuda_streams};
+use crate::high_level_api::global_state::with_thread_local_cuda_streams;
 #[cfg(feature = "gpu")]
 use crate::integer::gpu::ciphertext::CudaIntegerRadixCiphertext;
 use crate::Device;
@@ -213,9 +214,7 @@ impl RadixCiphertext {
     }
 
     #[inline]
-    #[allow(clippy::unused_self, clippy::needless_pass_by_ref_mut)]
     pub(crate) fn move_to_device_of_server_key_if_set(&mut self) {
-        #[cfg(feature = "gpu")]
         if let Some(device) = global_state::device_of_internal_keys() {
             self.move_to_device(device);
         }