Merge pull request #6174 from jferrant/chore/disallow-unused-imports-stackslib

Remove allow unused imports and fix clippy throughout stackslib

Author: obycode

stackslib/src/burnchains/affirmation.rs

@@ -231,33 +231,18 @@
 /// bootstrapping nodes.
 ///
 use std::cmp;
-use std::collections::{BTreeMap, HashMap, HashSet, VecDeque};
+use std::collections::{BTreeMap, HashMap, HashSet};
 use std::fmt;
 use std::fmt::Write;
-use std::sync::mpsc::SyncSender;
-use std::time::Duration;
 
 use serde::de::Error as de_Error;
-use serde::ser::Error as ser_Error;
 use serde::{Deserialize, Serialize};
-use stacks_common::types::chainstate::{
-    BlockHeaderHash, BurnchainHeaderHash, PoxId, SortitionId, StacksAddress, StacksBlockId,
-};
-
-use crate::burnchains::db::{
-    BurnchainBlockData, BurnchainDB, BurnchainDBTransaction, BurnchainHeaderReader,
-};
-use crate::burnchains::{Address, Burnchain, BurnchainBlockHeader, Error, PoxConstants, Txid};
-use crate::chainstate::burn::db::sortdb::SortitionDB;
-use crate::chainstate::burn::operations::leader_block_commit::{
-    RewardSetInfo, BURN_BLOCK_MINED_AT_MODULUS,
-};
+
+use crate::burnchains::db::{BurnchainDB, BurnchainDBTransaction, BurnchainHeaderReader};
+use crate::burnchains::{Burnchain, Error, PoxConstants};
+use crate::chainstate::burn::operations::leader_block_commit::BURN_BLOCK_MINED_AT_MODULUS;
 use crate::chainstate::burn::operations::{BlockstackOperationType, LeaderBlockCommitOp};
-use crate::chainstate::burn::{BlockSnapshot, ConsensusHash};
-use crate::chainstate::stacks::StacksBlockHeader;
-use crate::core::StacksEpochId;
-use crate::util_lib::boot::boot_code_id;
-use crate::util_lib::db::{DBConn, Error as DBError};
+use crate::util_lib::db::Error as DBError;
 
 /// Affirmation map entries.  By building on a PoX-mined block,
 /// a PoB-mined block (in a PoX reward cycle),

stackslib/src/burnchains/bitcoin/address.rs

@@ -15,14 +15,14 @@
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 use stacks_common::address::b58 as base58;
-use stacks_common::address::c32::c32_address;
 use stacks_common::deps_common::bech32;
 use stacks_common::deps_common::bech32::{u5, FromBase32, ToBase32};
 use stacks_common::deps_common::bitcoin::blockdata::opcodes::All as BtcOp;
 use stacks_common::deps_common::bitcoin::blockdata::script::Builder as BtcScriptBuilder;
 use stacks_common::deps_common::bitcoin::blockdata::transaction::TxOut;
-use stacks_common::util::hash::{hex_bytes, to_hex, Hash160};
-use stacks_common::util::log;
+#[cfg(test)]
+use stacks_common::util::hash::hex_bytes;
+use stacks_common::util::hash::Hash160;
 
 use crate::burnchains::bitcoin::{BitcoinNetworkType, Error as btc_error};
 use crate::burnchains::Address;
@@ -686,7 +686,6 @@ impl Address for BitcoinAddress {
 mod tests {
     use stacks_common::types::Address;
     use stacks_common::util::hash::{hex_bytes, Hash160};
-    use stacks_common::util::log;
 
     use super::{
         BitcoinAddress, LegacyBitcoinAddress, LegacyBitcoinAddressType, SegwitBitcoinAddress,

stackslib/src/burnchains/bitcoin/bits.rs

@@ -13,30 +13,21 @@
 //
 // You should have received a copy of the GNU General Public License
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-use sha2::{Digest, Sha256};
-use stacks_common::address::{public_keys_to_address_hash, AddressHashMode};
 use stacks_common::deps_common::bitcoin::blockdata::opcodes::{All as btc_opcodes, Class};
-use stacks_common::deps_common::bitcoin::blockdata::script::{Builder, Instruction, Script};
+use stacks_common::deps_common::bitcoin::blockdata::script::{Instruction, Script};
 use stacks_common::deps_common::bitcoin::blockdata::transaction::{
     TxIn as BtcTxIn, TxOut as BtcTxOut,
 };
-use stacks_common::deps_common::bitcoin::util::hash::Sha256dHash;
-use stacks_common::types::chainstate::BurnchainHeaderHash;
-use stacks_common::util::hash::{hex_bytes, Hash160};
-use stacks_common::util::log;
+#[cfg(test)]
+use stacks_common::util::hash::hex_bytes;
 
 use crate::burnchains::bitcoin::address::{BitcoinAddress, LegacyBitcoinAddressType};
 use crate::burnchains::bitcoin::keys::BitcoinPublicKey;
 use crate::burnchains::bitcoin::{
     BitcoinInputType, BitcoinNetworkType, BitcoinTxInput, BitcoinTxInputRaw,
     BitcoinTxInputStructured, BitcoinTxOutput, Error as btc_error,
 };
-use crate::burnchains::{PublicKey, Txid};
-use crate::chainstate::stacks::{
-    C32_ADDRESS_VERSION_MAINNET_MULTISIG, C32_ADDRESS_VERSION_MAINNET_SINGLESIG,
-    C32_ADDRESS_VERSION_TESTNET_MULTISIG, C32_ADDRESS_VERSION_TESTNET_SINGLESIG,
-};
+use crate::burnchains::Txid;
 
 /// Parse a script into its structured constituant opcodes and data and collect them
 pub fn parse_script(script: &Script) -> Vec<Instruction<'_>> {
@@ -624,14 +615,12 @@ mod tests {
     use stacks_common::deps_common::bitcoin::blockdata::transaction::Transaction;
     use stacks_common::deps_common::bitcoin::network::serialize::deserialize as bitcoinlib_deserialize;
     use stacks_common::util::hash::hex_bytes;
-    use stacks_common::util::log;
 
     use super::{
-        parse_script, to_txid, BitcoinTxInput, BitcoinTxInputRaw, BitcoinTxInputStructured,
-        BitcoinTxOutput,
+        parse_script, to_txid, BitcoinTxInputRaw, BitcoinTxInputStructured, BitcoinTxOutput,
     };
     use crate::burnchains::bitcoin::address::{
-        BitcoinAddress, LegacyBitcoinAddress, LegacyBitcoinAddressType, SegwitBitcoinAddress,
+        BitcoinAddress, LegacyBitcoinAddressType, SegwitBitcoinAddress,
     };
     use crate::burnchains::bitcoin::keys::BitcoinPublicKey;
     use crate::burnchains::bitcoin::{BitcoinInputType, BitcoinNetworkType};

stackslib/src/burnchains/bitcoin/blocks.rs

@@ -14,8 +14,6 @@
 // You should have received a copy of the GNU General Public License
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-use std::ops::Deref;
-
 use stacks_common::deps_common::bitcoin::blockdata::block::{Block, LoneBlockHeader};
 use stacks_common::deps_common::bitcoin::blockdata::opcodes::All as btc_opcodes;
 use stacks_common::deps_common::bitcoin::blockdata::script::{Instruction, Script};
@@ -25,25 +23,21 @@ use stacks_common::deps_common::bitcoin::network::serialize::BitcoinHash;
 use stacks_common::deps_common::bitcoin::util::hash::bitcoin_merkle_root;
 use stacks_common::types::chainstate::BurnchainHeaderHash;
 use stacks_common::util::hash::to_hex;
-use stacks_common::util::log;
 
 use crate::burnchains::bitcoin::address::BitcoinAddress;
 use crate::burnchains::bitcoin::indexer::BitcoinIndexer;
-use crate::burnchains::bitcoin::keys::BitcoinPublicKey;
 use crate::burnchains::bitcoin::messages::BitcoinMessageHandler;
 use crate::burnchains::bitcoin::{
-    bits, BitcoinBlock, BitcoinInputType, BitcoinNetworkType, BitcoinTransaction, BitcoinTxInput,
-    BitcoinTxOutput, Error as btc_error, PeerMessage,
+    bits, BitcoinBlock, BitcoinNetworkType, BitcoinTransaction, BitcoinTxInput, BitcoinTxOutput,
+    Error as btc_error, PeerMessage,
 };
 use crate::burnchains::indexer::{
     BurnBlockIPC, BurnHeaderIPC, BurnchainBlockDownloader, BurnchainBlockParser,
 };
 use crate::burnchains::{
-    BurnchainBlock, BurnchainTransaction, Error as burnchain_error, MagicBytes, Txid,
-    MAGIC_BYTES_LENGTH,
+    BurnchainBlock, Error as burnchain_error, MagicBytes, Txid, MAGIC_BYTES_LENGTH,
 };
 use crate::core::StacksEpochId;
-use crate::deps;
 
 #[derive(Debug, Clone, PartialEq)]
 pub struct BitcoinHeaderIPC {
@@ -550,16 +544,15 @@ mod tests {
     use stacks_common::types::chainstate::BurnchainHeaderHash;
     use stacks_common::types::Address;
     use stacks_common::util::hash::hex_bytes;
-    use stacks_common::util::log;
 
     use super::BitcoinBlockParser;
     use crate::burnchains::bitcoin::address::{BitcoinAddress, LegacyBitcoinAddressType};
     use crate::burnchains::bitcoin::keys::BitcoinPublicKey;
     use crate::burnchains::bitcoin::{
-        BitcoinBlock, BitcoinInputType, BitcoinNetworkType, BitcoinTransaction, BitcoinTxInput,
-        BitcoinTxInputRaw, BitcoinTxInputStructured, BitcoinTxOutput,
+        BitcoinBlock, BitcoinInputType, BitcoinNetworkType, BitcoinTransaction, BitcoinTxInputRaw,
+        BitcoinTxInputStructured, BitcoinTxOutput,
     };
-    use crate::burnchains::{BurnchainBlock, BurnchainTransaction, MagicBytes, Txid};
+    use crate::burnchains::{MagicBytes, Txid};
     use crate::core::StacksEpochId;
 
     struct TxFixture {

stackslib/src/burnchains/bitcoin/indexer.rs

@@ -15,23 +15,24 @@
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 use std::net::Shutdown;
-use std::ops::{Deref, DerefMut};
 use std::path::PathBuf;
 use std::sync::atomic::{AtomicBool, Ordering};
 use std::sync::Arc;
 use std::time::Duration;
-use std::{cmp, fs, net, path, time};
+use std::{cmp, fs, net};
 
 use rand::{thread_rng, Rng};
+#[cfg(test)]
 use stacks_common::deps_common::bitcoin::blockdata::block::{BlockHeader, LoneBlockHeader};
+#[cfg(test)]
 use stacks_common::deps_common::bitcoin::network::encodable::VarInt;
 use stacks_common::deps_common::bitcoin::network::message::NetworkMessage;
 use stacks_common::deps_common::bitcoin::network::serialize::{
     BitcoinHash, Error as btc_serialization_err,
 };
 use stacks_common::deps_common::bitcoin::util::hash::Sha256dHash;
 use stacks_common::types::chainstate::BurnchainHeaderHash;
-use stacks_common::util::{get_epoch_time_secs, log};
+use stacks_common::util::get_epoch_time_secs;
 
 use crate::burnchains::bitcoin::blocks::{
     BitcoinBlockDownloader, BitcoinBlockParser, BitcoinHeaderIPC,
@@ -42,7 +43,7 @@ use crate::burnchains::bitcoin::{BitcoinNetworkType, Error as btc_error};
 use crate::burnchains::db::BurnchainHeaderReader;
 use crate::burnchains::indexer::{BurnchainIndexer, *};
 use crate::burnchains::{
-    Burnchain, BurnchainBlockHeader, Error as burnchain_error, MagicBytes, BLOCKSTACK_MAGIC_MAINNET,
+    BurnchainBlockHeader, Error as burnchain_error, MagicBytes, BLOCKSTACK_MAGIC_MAINNET,
 };
 use crate::core::{
     EpochList, StacksEpoch, StacksEpochExtension, STACKS_EPOCHS_MAINNET, STACKS_EPOCHS_REGTEST,
@@ -1181,15 +1182,12 @@ mod test {
 
     use stacks_common::deps_common::bitcoin::blockdata::block::{BlockHeader, LoneBlockHeader};
     use stacks_common::deps_common::bitcoin::network::encodable::VarInt;
-    use stacks_common::deps_common::bitcoin::network::serialize::{
-        deserialize, serialize, BitcoinHash,
-    };
     use stacks_common::deps_common::bitcoin::util::hash::Sha256dHash;
     use stacks_common::util::get_epoch_time_secs;
     use stacks_common::util::uint::Uint256;
 
     use super::*;
-    use crate::burnchains::bitcoin::{Error as btc_error, *};
+    use crate::burnchains::bitcoin::*;
     use crate::burnchains::{Error as burnchain_error, *};
 
     #[test]

stackslib/src/burnchains/bitcoin/mod.rs

@@ -17,7 +17,6 @@
 // This module is concerned with the implementation of the BitcoinIndexer
 // structure and its methods and traits.
 
-use std::sync::Arc;
 use std::{error, fmt, io};
 
 use stacks_common::deps_common::bitcoin::network::serialize::Error as btc_serialize_error;
@@ -27,8 +26,6 @@ use stacks_common::util::HexError as btc_hex_error;
 use crate::burnchains::bitcoin::address::BitcoinAddress;
 use crate::burnchains::bitcoin::keys::BitcoinPublicKey;
 use crate::burnchains::Txid;
-use crate::chainstate::burn::operations::BlockstackOperationType;
-use crate::deps;
 use crate::util_lib::db::Error as db_error;
 
 pub mod address;

stackslib/src/burnchains/bitcoin/network.rs

@@ -16,7 +16,6 @@
 
 use std::io::Write;
 use std::net::SocketAddr;
-use std::ops::Deref;
 use std::sync::atomic::Ordering;
 use std::time::{SystemTime, UNIX_EPOCH};
 use std::{io, thread, time};
@@ -32,7 +31,7 @@ use stacks_common::deps_common::bitcoin::network::{
     serialize as btc_serialize,
 };
 use stacks_common::deps_common::bitcoin::util::hash::Sha256dHash;
-use stacks_common::util::{get_epoch_time_secs, log};
+use stacks_common::util::get_epoch_time_secs;
 
 use crate::burnchains::bitcoin::indexer::{network_id_to_bytes, BitcoinIndexer};
 use crate::burnchains::bitcoin::messages::BitcoinMessageHandler;

stackslib/src/burnchains/bitcoin/spv.rs

@@ -15,33 +15,27 @@
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 use std::collections::VecDeque;
-use std::io::{Read, Seek, SeekFrom, Write};
-use std::ops::Deref;
 use std::{cmp, fs};
 
-use rusqlite::types::{FromSql, FromSqlError, FromSqlResult, ToSql, ToSqlOutput, ValueRef};
-use rusqlite::{params, Connection, OpenFlags, OptionalExtension, Row, Transaction};
+use rusqlite::{params, OpenFlags, OptionalExtension, Row, Transaction};
 use stacks_common::deps_common::bitcoin::blockdata::block::{BlockHeader, LoneBlockHeader};
 use stacks_common::deps_common::bitcoin::blockdata::constants::genesis_block;
 use stacks_common::deps_common::bitcoin::network::constants::Network;
 use stacks_common::deps_common::bitcoin::network::encodable::VarInt;
 use stacks_common::deps_common::bitcoin::network::message as btc_message;
-use stacks_common::deps_common::bitcoin::network::serialize::{
-    deserialize, serialize, BitcoinHash,
-};
+use stacks_common::deps_common::bitcoin::network::serialize::BitcoinHash;
 use stacks_common::deps_common::bitcoin::util::hash::Sha256dHash;
 use stacks_common::types::chainstate::BurnchainHeaderHash;
 use stacks_common::types::sqlite::NO_PARAMS;
-use stacks_common::util::hash::{hex_bytes, to_hex};
+use stacks_common::util::get_epoch_time_secs;
 use stacks_common::util::uint::Uint256;
-use stacks_common::util::{get_epoch_time_secs, log};
 
 use crate::burnchains::bitcoin::indexer::BitcoinIndexer;
 use crate::burnchains::bitcoin::messages::BitcoinMessageHandler;
 use crate::burnchains::bitcoin::{BitcoinNetworkType, Error as btc_error, PeerMessage};
 use crate::util_lib::db::{
-    query_int, query_row, query_rows, sqlite_open, tx_begin_immediate, tx_busy_handler, u64_to_sql,
-    DBConn, DBTx, Error as db_error, FromColumn, FromRow,
+    query_row, sqlite_open, tx_begin_immediate, u64_to_sql, DBConn, DBTx, Error as db_error,
+    FromColumn, FromRow,
 };
 
 const BLOCK_HEADER_SIZE: u64 = 81;
@@ -1277,14 +1271,12 @@ impl BitcoinMessageHandler for SpvClient {
 mod test {
 
     use std::env;
-    use std::fs::*;
 
     use stacks_common::deps_common::bitcoin::blockdata::block::{BlockHeader, LoneBlockHeader};
     use stacks_common::deps_common::bitcoin::network::serialize::{
         deserialize, serialize, BitcoinHash,
     };
     use stacks_common::deps_common::bitcoin::util::hash::Sha256dHash;
-    use stacks_common::util::log;
 
     use super::*;
     use crate::burnchains::bitcoin::{Error as btc_error, *};