Skip to content

Commit

Permalink
fix: remove Default on Signature
Browse files Browse the repository at this point in the history
  • Loading branch information
leruaa committed Sep 8, 2024
1 parent 9807619 commit b6844b9
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions crates/primitives/src/signature/sig.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const SECP256K1N_ORDER: U256 =
uint!(0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFEBAAEDCE6AF48A03BBFD25E8CD0364141_U256);

/// An Ethereum ECDSA signature.
#[derive(Clone, Copy, Debug, Hash, Default, PartialEq, Eq)]
#[derive(Clone, Copy, Debug, Hash, PartialEq, Eq)]
pub struct Signature {
v: Parity,
r: U256,
Expand Down Expand Up @@ -789,14 +789,14 @@ mod tests {
#[cfg(feature = "rlp")]
#[test]
fn test_rlp_vrs_len() {
let signature = Signature::default();
assert_eq!(3, signature.rlp_vrs_len());
let signature = Signature::test_signature();
assert_eq!(67, signature.rlp_vrs_len());
}

#[cfg(feature = "rlp")]
#[test]
fn test_encode_and_decode() {
let signature = Signature::default();
let signature = Signature::test_signature();

let mut encoded = Vec::new();
signature.encode(&mut encoded);
Expand Down

0 comments on commit b6844b9

Please sign in to comment.