remove itertools

gitoxide-core/Cargo.toml

@@ -18,7 +18,7 @@ default = []
 ## Discover all git repositories within a directory. Particularly useful with [skim](https://github.com/lotabout/skim).
 organize = ["dep:gix-url", "dep:jwalk"]
 ## Derive the amount of time invested into a git repository akin to [git-hours](https://github.com/kimmobrunfeldt/git-hours).
-estimate-hours = ["dep:itertools", "dep:fs-err", "dep:crossbeam-channel", "dep:smallvec"]
+estimate-hours = ["dep:fs-err", "dep:crossbeam-channel", "dep:smallvec"]
 ## Gather information about repositories and store it in a database for easy querying.
 query = ["dep:rusqlite"]
 ## Run algorithms on a corpus of repositories and store their results for later comparison and intelligence gathering.
@@ -29,7 +29,7 @@ corpus = [ "dep:rusqlite", "dep:sysinfo", "organize", "dep:crossbeam-channel", "
 archive = ["dep:gix-archive-for-configuration-only", "gix/worktree-archive"]
 
 ## The ability to clean a repository, similar to `git clean`.
-clean = [ "gix/dirwalk" ]
+clean = ["gix/dirwalk"]
 
 #! ### Mutually Exclusive Networking
 #! If both are set, _blocking-client_ will take precedence, allowing `--all-features` to be used.
@@ -72,7 +72,6 @@ gix-url = { version = "^0.27.0", path = "../gix-url", optional = true }
 jwalk = { version = "0.8.0", optional = true }
 
 # for 'hours'
-itertools = { version = "0.12.0", optional = true }
 fs-err = { version = "2.6.0", optional = true }
 crossbeam-channel = { version = "0.5.6", optional = true }
 smallvec = { version = "1.10.0", optional = true }

gitoxide-core/src/hours/core.rs

@@ -7,7 +7,6 @@ use std::{
 };
 
 use gix::bstr::BStr;
-use itertools::Itertools;
 
 use crate::hours::{
     util::{add_lines, remove_lines},
@@ -25,17 +24,25 @@ pub fn estimate_hours(
     const MAX_COMMIT_DIFFERENCE_IN_MINUTES: f32 = 2.0 * MINUTES_PER_HOUR;
     const FIRST_COMMIT_ADDITION_IN_MINUTES: f32 = 2.0 * MINUTES_PER_HOUR;
 
-    let hours_for_commits = commits.iter().map(|t| &t.1).rev().tuple_windows().fold(
-        0_f32,
-        |hours, (cur, next): (&gix::actor::SignatureRef<'_>, &gix::actor::SignatureRef<'_>)| {
+    let hours_for_commits = {
+        let mut hours = 0.0;
+
+        let mut commits = commits.iter().map(|t| &t.1).rev();
+        let mut cur = commits.next().expect("not a single commit found");
+
+        for next in commits {
             let change_in_minutes = (next.time.seconds.saturating_sub(cur.time.seconds)) as f32 / MINUTES_PER_HOUR;
             if change_in_minutes < MAX_COMMIT_DIFFERENCE_IN_MINUTES {
-                hours + change_in_minutes / MINUTES_PER_HOUR
+                hours += change_in_minutes / MINUTES_PER_HOUR
             } else {
-                hours + (FIRST_COMMIT_ADDITION_IN_MINUTES / MINUTES_PER_HOUR)
+                hours += FIRST_COMMIT_ADDITION_IN_MINUTES / MINUTES_PER_HOUR
             }
-        },
-    );
+
+            cur = next;
+        }
+
+        hours
+    };
 
     let author = &commits[0].1;
     let (files, lines) = (!stats.is_empty())

gitoxide-core/src/hours/util.rs

@@ -1,7 +1,6 @@
 use std::sync::atomic::{AtomicUsize, Ordering};
 
 use gix::bstr::{BStr, ByteSlice};
-use itertools::Itertools;
 
 use crate::hours::core::HOURS_PER_WORKDAY;
 
@@ -53,9 +52,23 @@ impl WorkByPerson {
     ) -> std::io::Result<()> {
         writeln!(
             out,
-            "{} <{}>",
-            self.name.iter().join(", "),
-            self.email.iter().join(", ")
+            "{names} <{mails}>",
+            names = self
+                .name
+                .iter()
+                // BStr does not impl slice::Join
+                .map(|s| s.to_string())
+                .collect::<Vec<_>>()
+                .as_slice()
+                .join(", "),
+            mails = self
+                .email
+                .iter()
+                // BStr does not impl slice::Join
+                .map(|s| s.to_string())
+                .collect::<Vec<_>>()
+                .as_slice()
+                .join(", ")
         )?;
         writeln!(out, "{} commits found", self.num_commits)?;
         writeln!(