Apply suggestions from code review

Co-authored-by: Jack Grigg <thestr4d@gmail.com>

zcash_client_backend/src/data_api/wallet.rs

@@ -46,7 +46,7 @@ use zcash_keys::{
     keys::{UnifiedFullViewingKey, UnifiedSpendingKey},
 };
 use zcash_protocol::{PoolType, ShieldedProtocol};
-use zip321::{self, Payment};
+use zip321::Payment;
 
 use super::InputSource;
 use crate::{

zcash_client_sqlite/src/error.rs

@@ -8,7 +8,6 @@ use zcash_address::ParseError;
 use zcash_client_backend::data_api::NoteFilter;
 use zcash_keys::keys::AddressGenerationError;
 use zcash_protocol::{consensus::BlockHeight, value::BalanceError, PoolType};
-use zip32;
 
 use crate::{wallet::commitment_tree, AccountUuid};
 

zcash_client_sqlite/src/lib.rs

@@ -75,7 +75,7 @@ use zcash_protocol::{
     value::Zatoshis,
     ShieldedProtocol,
 };
-use zip32::{self, fingerprint::SeedFingerprint, DiversifierIndex};
+use zip32::{fingerprint::SeedFingerprint, DiversifierIndex};
 
 use crate::{error::SqliteClientError, wallet::commitment_tree::SqliteShardStore};
 
@@ -2257,7 +2257,6 @@ mod tests {
     pub(crate) fn fsblockdb_api() {
         use zcash_client_backend::data_api::testing::AddressType;
         use zcash_protocol::{consensus::NetworkConstants, value::Zatoshis};
-        use zip32;
 
         use crate::testing::FsBlockCache;
 

zcash_client_sqlite/src/wallet/init/migrations/receiving_key_scopes.rs

@@ -317,7 +317,7 @@ mod tests {
         memo::MemoBytes,
         value::Zatoshis,
     };
-    use zip32::{self, Scope};
+    use zip32::Scope;
 
     use crate::{
         error::SqliteClientError,

zcash_primitives/src/transaction/builder.rs

@@ -7,7 +7,6 @@ use std::fmt;
 use std::sync::mpsc::Sender;
 
 use ::sapling::{
-    self,
     builder::SaplingMetadata,
     prover::{OutputProver, SpendProver},
     Note, PaymentAddress,
@@ -994,10 +993,7 @@ mod testing {
     use rand::RngCore;
     use rand_core::CryptoRng;
 
-    use ::sapling::{
-        self,
-        prover::mock::{MockOutputProver, MockSpendProver},
-    };
+    use ::sapling::prover::mock::{MockOutputProver, MockSpendProver};
     use ::transparent::builder::TransparentSigningSet;
     use zcash_protocol::consensus;
 
@@ -1060,7 +1056,7 @@ mod tests {
     use super::{Builder, Error};
     use crate::transaction::builder::BuildConfig;
 
-    use ::sapling::{self, zip32::ExtendedSpendingKey, Node, Rseed};
+    use ::sapling::{zip32::ExtendedSpendingKey, Node, Rseed};
     use ::transparent::{address::TransparentAddress, builder::TransparentSigningSet};
     use zcash_protocol::{
         consensus::{NetworkUpgrade, Parameters, TEST_NETWORK},

zcash_primitives/src/transaction/mod.rs

@@ -18,7 +18,7 @@ use std::fmt::Debug;
 use std::io::{self, Read, Write};
 use std::ops::Deref;
 
-use ::sapling::{self, builder as sapling_builder};
+use ::sapling::builder as sapling_builder;
 use ::transparent::bundle::{self as transparent, OutPoint, TxIn, TxOut};
 use zcash_encoding::{CompactSize, Vector};
 use zcash_protocol::{

zcash_primitives/src/transaction/sighash.rs

@@ -4,7 +4,7 @@ use super::{
     sighash_v4::v4_signature_hash, sighash_v5::v5_signature_hash, Authorization, TransactionData,
     TxDigests, TxVersion,
 };
-use ::sapling::{self, bundle::GrothProofBytes};
+use ::sapling::bundle::GrothProofBytes;
 
 #[cfg(zcash_unstable = "zfuture")]
 use {crate::extensions::transparent::Precondition, zcash_protocol::value::Zatoshis};

zcash_primitives/src/transaction/sighash_v4.rs

@@ -1,10 +1,7 @@
 use blake2b_simd::{Hash as Blake2bHash, Params as Blake2bParams};
 use ff::PrimeField;
 
-use ::sapling::{
-    self,
-    bundle::{GrothProofBytes, OutputDescription, SpendDescription},
-};
+use ::sapling::bundle::{GrothProofBytes, OutputDescription, SpendDescription};
 use ::transparent::{
     bundle::{self as transparent, TxIn, TxOut},
     sighash::{SIGHASH_ANYONECANPAY, SIGHASH_MASK, SIGHASH_NONE, SIGHASH_SINGLE},

zcash_primitives/src/transaction/tests.rs

@@ -8,7 +8,6 @@ use ::transparent::{address::Script, sighash::TransparentAuthorizingContext};
 use zcash_protocol::{consensus::BranchId, value::Zatoshis};
 
 use super::{
-    sapling,
     sighash::SignableInput,
     sighash_v4::v4_signature_hash,
     sighash_v5::v5_signature_hash,

zcash_primitives/src/transaction/txid.rs

@@ -13,10 +13,7 @@ use zcash_protocol::{
     value::ZatBalance,
 };
 
-use ::sapling::{
-    self,
-    bundle::{OutputDescription, SpendDescription},
-};
+use ::sapling::bundle::{OutputDescription, SpendDescription};
 
 use super::{
     Authorization, Authorized, TransactionDigest, TransparentDigests, TxDigests, TxId, TxVersion,