From 0743f9e815fcb3824c188a97936baae90b9c6458 Mon Sep 17 00:00:00 2001
From: Kris Nuttycombe <kris.nuttycombe@gmail.com>
Date: Wed, 2 Jun 2021 17:11:04 -0600
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: str4d <jack@electriccoin.co>
---
 zcash_primitives/src/transaction/components/sapling/builder.rs | 1 +
 zcash_primitives/src/transaction/mod.rs                        | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/zcash_primitives/src/transaction/components/sapling/builder.rs b/zcash_primitives/src/transaction/components/sapling/builder.rs
index 91af3d8e3..5aaeaaf63 100644
--- a/zcash_primitives/src/transaction/components/sapling/builder.rs
+++ b/zcash_primitives/src/transaction/components/sapling/builder.rs
@@ -225,6 +225,7 @@ impl<P: consensus::Parameters> SaplingBuilder<P> {
         }
     }
 
+    /// Returns the net value represented by the spends and outputs added to this builder.
     pub fn value_balance(&self) -> Amount {
         self.value_balance
     }
diff --git a/zcash_primitives/src/transaction/mod.rs b/zcash_primitives/src/transaction/mod.rs
index 474f2d51c..11626bab3 100644
--- a/zcash_primitives/src/transaction/mod.rs
+++ b/zcash_primitives/src/transaction/mod.rs
@@ -181,7 +181,7 @@ impl TxVersion {
             BranchId::Sapling | BranchId::Blossom | BranchId::Heartwood | BranchId::Canopy => {
                 TxVersion::Sapling
             }
-            BranchId::Nu5 => TxVersion::ZFuture,
+            BranchId::Nu5 => unimplemented!(),
             #[cfg(feature = "zfuture")]
             BranchId::ZFuture => TxVersion::ZFuture,
         }