Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(primitives): Add y_parity_byte_non_eip155 to Parity #499

Merged
merged 4 commits into from
Jan 25, 2024
Merged

feat(primitives): Add y_parity_byte_non_eip155 to Parity #499

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7352343
feat(primitives): add fn to get v as 27,28 in the case of a non eip15…
Evalir Jan 24, 2024
f24a9a1
rename
Evalir Jan 24, 2024
50ebf49
Update crates/primitives/src/signature/parity.rs
Evalir Jan 25, 2024
9a923d5
feat: also apply + 27 in the simple Parity case
Evalir Jan 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions crates/primitives/src/signature/parity.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,15 @@ impl Parity {
self.y_parity() as u8
}

/// Return the y-parity byte as 27 or 28,
/// in the case of a non-EIP155 signature.
pub const fn y_parity_byte_non_eip155(&self) -> Option<u8> {
match self {
Self::NonEip155(v) | Self::Parity(v) => Some(*v as u8 + 27),
_ => None,
}
}

/// Inverts the parity.
pub const fn inverted(&self) -> Self {
match *self {
Expand Down
Loading