Qortal/pirate-librustzcash
3
0
Fork 0
mirror of https://github.com/Qortal/pirate-librustzcash.git synced 2025-02-11 17:55:46 +00:00
Code Issues Projects Releases Wiki Activity

Pass &impl TxProver to Builder::build

Browse Source 
This allows the caller to build multiple transactions with a single
proving backend.
This commit is contained in:
Jack Grigg 2020-02-08 00:37:31 +00:00
parent 76e0f658c1
commit 2d30c29d06
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
zcash_primitives/src/transaction/builder.rs
View File

@ -435,7 +435,7 @@ impl<R: RngCore + CryptoRng> Builder<R> {
pub fn build( pub fn build(
mut self, mut self,
consensus_branch_id: consensus::BranchId, consensus_branch_id: consensus::BranchId,
prover: impl TxProver, prover: &impl TxProver,
) -> Result<(Transaction, TransactionMetadata), Error> { ) -> Result<(Transaction, TransactionMetadata), Error> {
let mut tx_metadata = TransactionMetadata::new(); let mut tx_metadata = TransactionMetadata::new();
@ -558,7 +558,7 @@ impl<R: RngCore + CryptoRng> Builder<R> {
// Record the post-randomized output location // Record the post-randomized output location
tx_metadata.output_indices[pos] = i; tx_metadata.output_indices[pos] = i;
output.build(&prover, &mut ctx, &mut self.rng) output.build(prover, &mut ctx, &mut self.rng)
} else { } else {
// This is a dummy output // This is a dummy output
let (dummy_to, dummy_note) = { let (dummy_to, dummy_note) = {
@ -718,7 +718,7 @@ mod tests {
{ {
let builder = Builder::new(0); let builder = Builder::new(0);
assert_eq!( assert_eq!(
builder.build(consensus::BranchId::Sapling, MockTxProver), builder.build(consensus::BranchId::Sapling, &MockTxProver),
Err(Error::ChangeIsNegative(Amount::from_i64(-10000).unwrap())) Err(Error::ChangeIsNegative(Amount::from_i64(-10000).unwrap()))
); );
} }
@ -740,7 +740,7 @@ mod tests {
) )
.unwrap(); .unwrap();
assert_eq!( assert_eq!(
builder.build(consensus::BranchId::Sapling, MockTxProver), builder.build(consensus::BranchId::Sapling, &MockTxProver),
Err(Error::ChangeIsNegative(Amount::from_i64(-60000).unwrap())) Err(Error::ChangeIsNegative(Amount::from_i64(-60000).unwrap()))
); );
} }
@ -756,7 +756,7 @@ mod tests {
) )
.unwrap(); .unwrap();
assert_eq!( assert_eq!(
builder.build(consensus::BranchId::Sapling, MockTxProver), builder.build(consensus::BranchId::Sapling, &MockTxProver),
Err(Error::ChangeIsNegative(Amount::from_i64(-60000).unwrap())) Err(Error::ChangeIsNegative(Amount::from_i64(-60000).unwrap()))
); );
} }
@ -796,7 +796,7 @@ mod tests {
) )
.unwrap(); .unwrap();
assert_eq!( assert_eq!(
builder.build(consensus::BranchId::Sapling, MockTxProver), builder.build(consensus::BranchId::Sapling, &MockTxProver),
Err(Error::ChangeIsNegative(Amount::from_i64(-1).unwrap())) Err(Error::ChangeIsNegative(Amount::from_i64(-1).unwrap()))
); );
} }
@ -835,7 +835,7 @@ mod tests {
) )
.unwrap(); .unwrap();
assert_eq!( assert_eq!(
builder.build(consensus::BranchId::Sapling, MockTxProver), builder.build(consensus::BranchId::Sapling, &MockTxProver),
Err(Error::BindingSig) Err(Error::BindingSig)
) )
} }