zecfoundation
/
zebra
mirror of https://github.com/ZcashFoundation/zebra.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Remove .expect()s for block and transaction, they might fail for writer reasons

This commit is contained in:
Deirdre Connolly 2019-10-09 22:16:13 -04:00 committed by Deirdre Connolly
parent e1f2eaa446
commit dc18e8f24c
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
zebra-network/src/protocol/codec.rs
View File

@ -211,9 +211,7 @@ impl Codec {
ref block,
} => {
writer.write_u32::<LittleEndian>(version.0)?;
block
.zcash_serialize(&mut writer)
.expect("Blocks must serialize.");
block.zcash_serialize(&mut writer)?
}
GetBlocks {
ref version,
@ -268,9 +266,7 @@ impl Codec {
ref transaction,
} => {
writer.write_u32::<LittleEndian>(version.0)?;
transaction
.zcash_serialize(&mut writer)
.expect("Transactions must serialize.");
transaction.zcash_serialize(&mut writer)?
}
// Mempool => {}
// FilterLoad => {}