satisfy clippy

common-api/src/crypto/ed25519.rs

@@ -80,7 +80,7 @@ impl From<&Ed25519SecretKey> for Ed25519PublicKey {
 
 // This `Hash` implementation is safe since it retains the property
 // `k1 == k2 ⇒ hash(k1) == hash(k2)`.
-#[allow(clippy::derive_hash_xor_eq)]
+#[allow(clippy::derived_hash_with_manual_eq)]
 impl Hash for Ed25519PublicKey {
     fn hash<H: Hasher>(&self, state: &mut H) {
         state.write_u8(0u8);
@@ -197,7 +197,7 @@ impl BorshSerialize for Ed25519Signature {
     }
 }
 
-#[allow(clippy::derive_hash_xor_eq)]
+#[allow(clippy::derived_hash_with_manual_eq)]
 impl Hash for Ed25519Signature {
     fn hash<H: Hasher>(&self, state: &mut H) {
         self.to_bytes().hash(state);

near-primitives-light/src/transaction.rs

@@ -257,9 +257,10 @@ impl Borrow<CryptoHash> for SignedTransaction {
 }
 
 /// The status of execution for a transaction or a receipt.
-#[derive(BorshSerialize, BorshDeserialize, PartialEq, Eq, Clone)]
+#[derive(Default, BorshSerialize, BorshDeserialize, PartialEq, Eq, Clone)]
 pub enum ExecutionStatus {
     /// The execution is pending or unknown.
+    #[default]
     Unknown,
     /// The execution has failed with the given execution error.
     Failure(Box<TxExecutionError>),
@@ -285,12 +286,6 @@ impl fmt::Debug for ExecutionStatus {
     }
 }
 
-impl Default for ExecutionStatus {
-    fn default() -> Self {
-        ExecutionStatus::Unknown
-    }
-}
-
 /// ExecutionOutcome for proof. Excludes logs and metadata
 #[derive(BorshSerialize, BorshDeserialize, PartialEq, Eq, Clone)]
 pub struct PartialExecutionOutcome {
@@ -371,21 +366,16 @@ impl Default for ExecutionOutcome {
     }
 }
 
-#[derive(BorshSerialize, BorshDeserialize, PartialEq, Clone, Eq, Debug)]
+#[derive(Default, BorshSerialize, BorshDeserialize, PartialEq, Clone, Eq, Debug)]
 pub enum ExecutionMetadata {
     // V1: Empty Metadata
+    #[default]
     V1,
 
     // V2: With ProfileData
     V2(ProfileData),
 }
 
-impl Default for ExecutionMetadata {
-    fn default() -> Self {
-        ExecutionMetadata::V1
-    }
-}
-
 impl fmt::Debug for ExecutionOutcome {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         f.debug_struct("ExecutionOutcome")

near-primitives-light/src/types.rs

@@ -12,22 +12,17 @@ use serde::{Deserialize, Serialize};
 pub type StateRoot = CryptoHash;
 
 /// Different types of finality.
-#[derive(Serialize, Deserialize, Clone, Debug, PartialEq, Eq)]
+#[derive(Default, Serialize, Deserialize, Clone, Debug, PartialEq, Eq)]
 pub enum Finality {
     #[serde(rename = "optimistic")]
     None,
     #[serde(rename = "near-final")]
     DoomSlug,
+    #[default]
     #[serde(rename = "final")]
     Final,
 }
 
-impl Default for Finality {
-    fn default() -> Self {
-        Finality::Final
-    }
-}
-
 #[derive(Debug, Serialize, Deserialize)]
 pub struct AccountWithPublicKey {
     pub account_id: AccountId,

near-primitives-light/src/views.rs

@@ -596,9 +596,12 @@ impl From<SignedTransaction> for SignedTransactionView {
 }
 
 #[allow(clippy::large_enum_variant)]
-#[derive(BorshSerialize, BorshDeserialize, Serialize, Deserialize, PartialEq, Eq, Clone)]
+#[derive(
+    Default, BorshSerialize, BorshDeserialize, Serialize, Deserialize, PartialEq, Eq, Clone,
+)]
 pub enum FinalExecutionStatus {
     /// The execution has not yet started.
+    #[default]
     NotStarted,
     /// The execution has started and still going.
     Started,
@@ -621,12 +624,6 @@ impl fmt::Debug for FinalExecutionStatus {
     }
 }
 
-impl Default for FinalExecutionStatus {
-    fn default() -> Self {
-        FinalExecutionStatus::NotStarted
-    }
-}
-
 #[allow(clippy::large_enum_variant)]
 #[derive(BorshSerialize, BorshDeserialize, Serialize, Deserialize, Debug, PartialEq, Eq, Clone)]
 pub enum ServerError {