zec
/
librustzcash
mirror of https://github.com/zcash/librustzcash.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

zcash_client_backend: Box `zip321::parse::Param::Addr` 

After adding the `RecipientAddress::Unified` enum case, `Param::Addr`
is now much larger than the other `Param` cases. This change reduces the
size of the `Param` enum, fixing a clippy lint.
This commit is contained in:
Jack Grigg 2022-06-10 21:45:11 +00:00
parent 555a621a10
commit 97aee898b2
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
zcash_client_backend/src/zip321.rs
View File

@ -393,7 +393,7 @@ mod parse {
/// ZIP 321 URI.
#[derive(Debug, PartialEq)]
pub enum Param {
Addr(RecipientAddress),
Addr(Box<RecipientAddress>),
Amount(Amount),
Memo(MemoBytes),
Label(String),
@ -440,7 +440,7 @@ mod parse {
});
let mut payment = Payment {
recipient_address: addr.ok_or(Zip321Error::RecipientMissing(i))?,
recipient_address: *addr.ok_or(Zip321Error::RecipientMissing(i))?,
amount: Amount::zero(),
memo: None,
label: None,
@ -485,7 +485,7 @@ mod parse {
// then cause `map_opt` to fail.
RecipientAddress::decode(params, addr_str).map(|a| {
Some(IndexedParam {
param: Param::Addr(a),
param: Param::Addr(Box::new(a)),
payment_index: 0,
})
})
@ -589,6 +589,7 @@ mod parse {
) -> Result<IndexedParam, String> {
let param = match name {
"address" => RecipientAddress::decode(params, value)
.map(Box::new)
.map(Param::Addr)
.ok_or(format!(
"Could not interpret {} as a valid Zcash address.",