Fix all compiler warnings and clippy issues

- Remove unused re-exports from module files
- Change worker_func to take ConfigSettings by reference
- Collapse nested if-else blocks in files.rs
- Make parse functions public for unit tests
- Add #[cfg(test)] attribute to unit_tests module
- Remove unused PathBuf import from integration tests

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: lookbusy1344

src/cli/args.rs

@@ -5,7 +5,7 @@ use anyhow::{Result, anyhow};
 use pico_args::Arguments;
 
 use crate::cli::config::{ConfigSettings, HELP};
-use crate::core::types::{HashAlgorithm, OutputEncoding, DEFAULT_HASH, VERSION, GIT_VERSION_SHORT};
+use crate::core::types::{DEFAULT_HASH, GIT_VERSION_SHORT, HashAlgorithm, OutputEncoding, VERSION};
 
 pub fn process_command_line(mut pargs: Arguments) -> Result<ConfigSettings> {
     let algo_str: Option<String> = pargs.opt_value_from_str(["-a", "--algorithm"])?;
@@ -54,14 +54,14 @@ pub fn process_command_line(mut pargs: Arguments) -> Result<ConfigSettings> {
     Ok(config)
 }
 
-fn parse_hash_algorithm(algorithm: Option<&str>) -> Result<HashAlgorithm, strum::ParseError> {
+pub fn parse_hash_algorithm(algorithm: Option<&str>) -> Result<HashAlgorithm, strum::ParseError> {
     match algorithm {
         Some(algo_str) if !algo_str.is_empty() => HashAlgorithm::from_str(algo_str),
         _ => Ok(DEFAULT_HASH),
     }
 }
 
-fn parse_hash_encoding(encoding: Option<&str>) -> Result<OutputEncoding, strum::ParseError> {
+pub fn parse_hash_encoding(encoding: Option<&str>) -> Result<OutputEncoding, strum::ParseError> {
     match encoding {
         Some(enc_str) if !enc_str.is_empty() => OutputEncoding::from_str(enc_str),
         _ => Ok(OutputEncoding::Unspecified),
@@ -90,4 +90,4 @@ fn args_finished(args: Arguments) -> Result<Vec<OsString>> {
     }
 
     Ok(unused)
-}
+}

src/cli/config.rs

@@ -64,4 +64,4 @@ OPTIONS:
 Algorithm can be:
     CRC32, MD5, SHA1, WHIRLPOOL, BLAKE2S-256, BLAKE2B-512,
     SHA2 / SHA2-256 / SHA-256, SHA2-224, SHA2-384, SHA2-512,
-    SHA3 / SHA3-256 (default), SHA3-384, SHA3-512";
+    SHA3 / SHA3-256 (default), SHA3-384, SHA3-512";

src/cli/mod.rs

@@ -2,4 +2,3 @@ pub mod args;
 pub mod config;
 
 pub use args::{process_command_line, show_help};
-pub use config::{ConfigSettings, HELP};

src/core/mod.rs

@@ -1,5 +1,4 @@
 pub mod types;
 pub mod worker;
 
-pub use types::{BasicHash, HashAlgorithm, OutputEncoding, DEFAULT_HASH, VERSION, GIT_VERSION_SHORT};
-pub use worker::worker_func;
+pub use worker::worker_func;

src/core/types.rs

@@ -61,4 +61,4 @@ impl fmt::Display for BasicHash {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         write!(f, "{}", self.0)
     }
-}
+}

src/core/worker.rs

@@ -10,12 +10,12 @@ use crate::hash::algorithms::call_hasher;
 use crate::io::files::get_required_filenames;
 use crate::progress::ProgressManager;
 
-pub fn worker_func(config: ConfigSettings) -> Result<()> {
+pub fn worker_func(config: &ConfigSettings) -> Result<()> {
     if config.debug_mode {
-        show_initial_info(&config);
+        show_initial_info(config);
     }
 
-    let paths = get_required_filenames(&config)?;
+    let paths = get_required_filenames(config)?;
 
     if paths.is_empty() {
         if config.debug_mode {
@@ -29,9 +29,9 @@ pub fn worker_func(config: ConfigSettings) -> Result<()> {
     }
 
     if config.single_thread || paths.len() == 1 {
-        file_hashes_st(&config, &paths);
+        file_hashes_st(config, &paths);
     } else {
-        file_hashes_mt(&config, &paths);
+        file_hashes_mt(config, &paths);
     }
 
     Ok(())
@@ -145,4 +145,4 @@ where
     }
 
     result
-}
+}

src/hash/algorithms.rs

@@ -7,8 +7,8 @@ use sha3::{Sha3_256, Sha3_384, Sha3_512};
 use whirlpool::Whirlpool;
 
 use crate::core::types::{BasicHash, HashAlgorithm, OutputEncoding};
-use crate::hash::digest_impl::hash_file_encoded;
 use crate::hash::crc32::Crc32;
+use crate::hash::digest_impl::hash_file_encoded;
 
 pub fn call_hasher(
     algo: HashAlgorithm,
@@ -38,4 +38,4 @@ pub fn call_hasher(
         HashAlgorithm::Blake2S256 => hash_file_encoded::<Blake2s256>(path, encoding),
         HashAlgorithm::Blake2B512 => hash_file_encoded::<Blake2b512>(path, encoding),
     }
-}
+}

src/hash/crc32.rs

@@ -61,4 +61,4 @@ impl Write for Crc32 {
     fn flush(&mut self) -> std::io::Result<()> {
         Ok(())
     }
-}
+}

src/hash/digest_impl.rs

@@ -76,4 +76,4 @@ fn file_size(path: impl AsRef<str>) -> anyhow::Result<u64> {
 fn build_heap_buffer<T: Default + Copy>(len: usize) -> Box<[T]> {
     let vec = vec![T::default(); len];
     vec.into_boxed_slice()
-}
+}

src/hash/mod.rs

@@ -1,6 +1,3 @@
 pub mod algorithms;
 pub mod crc32;
 pub mod digest_impl;
-
-pub use algorithms::call_hasher;
-pub use digest_impl::hash_file_encoded;