fix: single file hash groups printed to screen

.cargo/config.toml

@@ -0,0 +1,2 @@
+[build]
+rustflags = ["-C", "target-feature=+aes,+sse2"]

Cargo.lock

@@ -273,7 +273,7 @@ dependencies = [
 
 [[package]]
 name = "deduplicator"
-version = "0.3.1"
+version = "0.3.2"
 dependencies = [
  "anyhow",
  "bytesize",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "deduplicator"
-version = "0.3.1"
+version = "0.3.2"
 edition = "2021"
 description = "find,filter and delete duplicate files"
 repository = "https://github.com/sreedevk/deduplicator"

README.md

@@ -155,6 +155,8 @@ dust 'bench_artifacts'
 ## proposed
 - [ ] parallelization
     - [ ] scanning + processing sw + processing hw + formatting + printing
+- [ ] user supplied cache file path for faster re-runs
+- [ ] hardlinks / symlinks support
 - [ ] max file path size should use the last set of duplicates
 - [ ] add more unit tests
     - [ ] test against different filesystems
@@ -177,6 +179,9 @@ dust 'bench_artifacts'
     - [ ] append the offset between the last initial page hashed and the first final page hashed in the content passed to the hasher.
 - [ ] fix: --exclude-types and --types flag behave identically.
 
+## v0.3.2
+- [x] fix: single file groups are printed to screen
+
 ## v0.3.1
 - [x] parallelization
     - [x] (scanning + processing sw + processing hw) & formatting & printing

src/formatter.rs

@@ -4,6 +4,7 @@ use chrono::{DateTime, Utc};
 use dashmap::DashMap;
 use pathdiff::diff_paths;
 use rayon::prelude::*;
+use std::sync::atomic::AtomicU64;
 use std::{path::PathBuf, sync::Arc};
 
 const YELLOW: &str = "\x1b[33m";
@@ -39,26 +40,34 @@ impl Formatter {
         if raw.is_empty() {
             println!("No duplicates found matching your search criteria.");
         } else {
+            let printed_count: AtomicU64 = AtomicU64::new(0);
+
             raw.par_iter().for_each(|sref| {
-                let mut ostring = format!("{}{:32x}{}\n", YELLOW, sref.key(), RESET);
-                let subfields = sref
-                    .value()
-                    .par_iter()
-                    .map(|finfo| {
-                        format!(
-                            "├─ {}\t{}\t{}\n",
-                            Self::human_path(finfo, aargs, max_path_len as usize)
-                                .expect("path formatting failed."),
-                            Self::human_filesize(finfo).expect("filesize formatting failed."),
-                            Self::human_mtime(finfo).expect("modified time formatting failed.")
-                        )
-                    })
-                    .collect::<String>();
+                if sref.value().len() > 1 {
+                    printed_count.fetch_add(1, std::sync::atomic::Ordering::Relaxed);
+                    let mut ostring = format!("{}{:32x}{}\n", YELLOW, sref.key(), RESET);
+                    let subfields = sref
+                        .value()
+                        .par_iter()
+                        .map(|finfo| {
+                            format!(
+                                "├─ {}\t{}\t{}\n",
+                                Self::human_path(finfo, aargs, max_path_len as usize)
+                                    .expect("path formatting failed."),
+                                Self::human_filesize(finfo).expect("filesize formatting failed."),
+                                Self::human_mtime(finfo).expect("modified time formatting failed.")
+                            )
+                        })
+                        .collect::<String>();
 
-                ostring.push_str(&subfields);
-
-                println!("{ostring}");
+                    ostring.push_str(&subfields);
+                    println!("{ostring}");
+                }
             });
+
+            if printed_count.load(std::sync::atomic::Ordering::Relaxed) < 1 {
+                println!("No duplicates found matching your search criteria.");
+            }
         }
     }
 }

src/interactive.rs

@@ -2,6 +2,7 @@ use crate::{fileinfo::FileInfo, formatter::Formatter, params::Params};
 use anyhow::Result;
 use dashmap::DashMap;
 use prettytable::{format, row, Table};
+use std::sync::atomic::AtomicU64;
 use std::{
     io::{self, Write},
     sync::Arc,
@@ -11,12 +12,19 @@ pub struct Interactive;
 
 impl Interactive {
     pub fn init(result: Arc<DashMap<u128, Vec<FileInfo>>>, app_args: &Params) -> Result<()> {
-        result
-            .clone()
+        let store = result.clone();
+        if store.is_empty() {
+            println!("No duplicates found matching your search criteria.");
+        }
+
+        let printed_count: AtomicU64 = AtomicU64::new(0);
+
+        store
             .iter()
             .filter(|i| i.value().len() > 1)
             .enumerate()
             .for_each(|(gindex, i)| {
+                printed_count.fetch_add(1, std::sync::atomic::Ordering::Relaxed);
                 let group = i.value();
                 let mut itable = Table::new();
                 itable.set_format(*format::consts::FORMAT_NO_BORDER_LINE_SEPARATOR);
@@ -40,6 +48,10 @@ impl Interactive {
                 Self::process_group_action(group, gindex, result.len(), itable);
             });
 
+        if printed_count.load(std::sync::atomic::Ordering::Relaxed) < 1 {
+            println!("No duplicates found matching your search criteria.");
+        }
+
         Ok(())
     }