fixing clippy stuff

Author: rbaconcordium

src/cis2/mod.rs

@@ -215,6 +215,7 @@ impl Cis2Contract {
     ///   update transaction.
     /// * `transaction_metadata` - Metadata for constructing the transaction.
     /// * `transfers` - A list of CIS2 token transfers to execute.
+    #[allow(clippy::result_large_err)]
     pub fn make_transfer(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -241,6 +242,7 @@ impl Cis2Contract {
 
     /// Like [`make_transfer`](Self::make_transfer), except it is more ergonomic
     /// when transferring a single token.
+    #[allow(clippy::result_large_err)]
     pub fn make_transfer_single(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -307,6 +309,7 @@ impl Cis2Contract {
     ///   update transaction.
     /// * `transaction_metadata` - Metadata for constructing the transaction.
     /// * `updates` - A list of CIS2 UpdateOperators to update.
+    #[allow(clippy::result_large_err)]
     pub fn make_update_operator(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -360,6 +363,7 @@ impl Cis2Contract {
 
     /// Like [`make_update_operator`](Self::make_update_operator), but more
     /// ergonomic when updating a single operator.
+    #[allow(clippy::result_large_err)]
     pub fn make_update_operator_single(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -489,6 +493,7 @@ impl Cis2Contract {
 
 /// Extract an element from the given vector if the vector has exactly one
 /// element. Otherwise raise a parse error.
+#[allow(clippy::result_large_err)]
 fn only_one<A, V: AsRef<Vec<A>>>(res: V) -> Result<A, Cis2QueryError>
 where
     Vec<A>: From<V>,

src/cis3.rs

@@ -189,6 +189,7 @@ impl Cis3Contract {
     /// * `params` - The parameters for the permit invocation. Includes the
     ///   signature of the sponsoree, the address of the sponsoree, and the
     ///   signed message.
+    #[allow(clippy::result_large_err)]
     pub fn make_permit(
         &mut self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -237,6 +238,7 @@ impl Cis3Contract {
 
 /// Extract an element from the given vector if the vector has exactly one
 /// element. Otherwise raise a parse error.
+#[allow(clippy::result_large_err)]
 fn only_one<A, V: AsRef<Vec<A>>>(res: V) -> Result<A, Cis3SupportsPermitError>
 where
     Vec<A>: From<V>,

src/cis4.rs

@@ -157,6 +157,7 @@ impl Cis4Contract {
 
     /// Construct a transaction for registering a new credential.
     /// Note that this **does not** send the transaction.c
+    #[allow(clippy::result_large_err)]
     pub fn make_register_credential(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -244,6 +245,7 @@ impl Cis4Contract {
     /// The extra nonce that must be provided is the holder's nonce inside the
     /// contract. The signature on this revocation message is set to expire at
     /// the same time as the transaction.
+    #[allow(clippy::result_large_err)]
     pub fn make_revoke_credential_as_holder(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,
@@ -310,6 +312,7 @@ impl Cis4Contract {
     /// key that signs the revocation message.
     /// The signature on this revocation message is set to expire at
     /// the same time as the transaction.
+    #[allow(clippy::result_large_err)]
     pub fn make_revoke_credential_other(
         &self,
         signer: &impl transactions::ExactSizeTransactionSigner,

src/protocol_level_tokens/mod.rs

@@ -23,7 +23,7 @@ impl TryFrom<generated::plt::TokenId> for TokenId {
 
     fn try_from(token_id: generated::plt::TokenId) -> Result<Self, Self::Error> {
         Self::try_from(token_id.value)
-            .map_err(|err| tonic::Status::internal(format!("Unexpected token identifier: {}", err)))
+            .map_err(|err| tonic::Status::internal(format!("Unexpected token identifier: {err}")))
     }
 }
 

src/v2/conversions.rs

@@ -37,8 +37,7 @@ fn consume<A: Deserial>(bytes: &[u8]) -> Result<A, tonic::Status> {
     match res {
         Ok(v) if cursor.position() == bytes.len() as u64 => Ok(v),
         Err(e) => Err(tonic::Status::internal(format!(
-            "Could not deserialize response: {}",
-            e
+            "Could not deserialize response: {e}"
         ))),
         Ok(_) => Err(tonic::Status::internal(
             "Could not deserialize response: trailing bytes",
@@ -48,6 +47,7 @@ fn consume<A: Deserial>(bytes: &[u8]) -> Result<A, tonic::Status> {
 
 /// Convert from the 0-based protobuf representation to the 1-based internal
 /// representation of a [ProtocolVersionInt].
+#[allow(clippy::result_large_err)]
 fn protocol_version_int_from_enum(tag_number: i32) -> Result<ProtocolVersionInt, tonic::Status> {
     if tag_number < 0 {
         Err(tonic::Status::internal(format!(
@@ -2273,7 +2273,7 @@ impl TryFrom<baker_event::Event> for super::types::BakerEvent {
             baker_event::Event::BakerSetMetadataUrl(v) => Self::BakerSetMetadataURL {
                 baker_id: v.baker_id.require()?.into(),
                 metadata_url: v.url.try_into().map_err(|e| {
-                    tonic::Status::invalid_argument(format!("Invalid argument: {}", e))
+                    tonic::Status::invalid_argument(format!("Invalid argument: {e}"))
                 })?,
             },
             baker_event::Event::BakerSetTransactionFeeCommission(v) => {
@@ -2317,7 +2317,7 @@ impl TryFrom<RegisteredData> for super::types::RegisteredData {
         value
             .value
             .try_into()
-            .map_err(|e| tonic::Status::invalid_argument(format!("{}", e)))
+            .map_err(|e| tonic::Status::invalid_argument(format!("{e}")))
     }
 }
 
@@ -3443,6 +3443,7 @@ impl TryFrom<block_special_event::AccountAmounts>
 {
     type Error = tonic::Status;
 
+    #[allow(clippy::result_large_err)]
     fn try_from(message: block_special_event::AccountAmounts) -> Result<Self, Self::Error> {
         fn mapper(
             entry: block_special_event::account_amounts::Entry,

src/v2/mod.rs

@@ -3010,6 +3010,7 @@ impl FinalizedBlocksStream {
     }
 }
 
+#[allow(clippy::result_large_err)]
 fn extract_metadata<T>(response: &tonic::Response<T>) -> endpoints::RPCResult<BlockHash> {
     match response.metadata().get("blockhash") {
         Some(bytes) => {