Implement From on u8 for RecoveryId instead of i32

tcharding · tcharding · commit 8ef905da38a5 · 2025-06-12T13:26:34.000+10:00
The `i32` is a hangover from before #743, now we have a nice enum and a `from_u8_masked` constructor (along with a `to_u8` getter) lets make the `From<RecoveryId>` impl be on `u8` instead of on `i32`.
diff --git a/src/ecdsa/recovery.rs b/src/ecdsa/recovery.rs
@@ -62,7 +62,7 @@ impl TryFrom<i32> for RecoveryId {
     }
 }
 
-impl From<RecoveryId> for i32 {
+impl From<RecoveryId> for u8 {
     #[inline]
     fn from(val: RecoveryId) -> Self {
         match val {

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ impl TryFrom<i32> for RecoveryId {`
`62`	`62`	`}`
`63`	`63`	`}`
`64`	`64`
`65`		`-impl From<RecoveryId> for i32 {`
	`65`	`+impl From<RecoveryId> for u8 {`
`66`	`66`	`#[inline]`
`67`	`67`	`fn from(val: RecoveryId) -> Self {`
`68`	`68`	`match val {`