Merge rust-bitcoin/rust-bitcoin#3488: refactor: compute recid in a single line
9d9a2d53e1
refactor: compute recid in a single line (ChrisCho-H)
Pull request description:
Just miscellaneous change that doesn't affect any but simpler and cleaner code.
ACKs for top commit:
apoelstra:
ACK 9d9a2d53e13d5de08440349e97b2eb393e7833dc; successfully ran local tests; will one-ACK merge; this seems like a clear improvement
Tree-SHA512: e90ccaad8878fdc0eeeced687f8734ad13d228a235d09be30021f7e967769ca11a4418f75e0d04b4a78aef19b37b658408f32ca6c11105252070af3a0f63709d
This commit is contained in:
commit
62db1dfec0
|
@ -101,11 +101,7 @@ mod message_signing {
|
|||
pub fn serialize(&self) -> [u8; 65] {
|
||||
let (recid, raw) = self.signature.serialize_compact();
|
||||
let mut serialized = [0u8; 65];
|
||||
serialized[0] = 27;
|
||||
serialized[0] += recid.to_i32() as u8;
|
||||
if self.compressed {
|
||||
serialized[0] += 4;
|
||||
}
|
||||
serialized[0] = recid.to_i32() as u8 + if self.compressed { 31 } else { 27 };
|
||||
serialized[1..].copy_from_slice(&raw[..]);
|
||||
serialized
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue