Merge branch 'optimizations'

Author: Byron

Cargo.lock

@@ -44,7 +44,7 @@ max = ["max-control", "fast", "gitoxide-core-blocking-client", "http-client-curl
 max-pure = ["max-control", "gix-features/rustsha1", "gix-features/zlib-rust-backend", "http-client-reqwest", "gitoxide-core-blocking-client" ]
 
 ## Like `max`, but with more control for configuration. See the *Package Maintainers* headline for more information.
-max-control = ["tracing", "fast-safe", "pretty-cli", "gitoxide-core-tools-query", "gitoxide-core-tools-corpus", "gitoxide-core-tools", "prodash-render-line", "prodash-render-tui", "prodash/render-line-autoconfigure", "gix/regex" ]
+max-control = ["tracing", "fast-safe", "pretty-cli", "gitoxide-core-tools-query", "gitoxide-core-tools-corpus", "gitoxide-core-tools", "prodash-render-line", "prodash-render-tui", "prodash/render-line-autoconfigure", "gix/revparse-regex" ]
 
 ## All of the good stuff, with less fanciness for smaller binaries.
 ##

Cargo.toml

@@ -44,8 +44,8 @@ serde = ["gix/serde", "dep:serde_json", "dep:serde", "bytesize/serde"]
 
 [dependencies]
 # deselect everything else (like "performance") as this should be controllable by the parent application.
-gix = { version = "^0.53.0", path = "../gix", default-features = false }
-gix-pack-for-configuration-only = { package = "gix-pack", version = "^0.41.0", path = "../gix-pack", default-features = false, features = ["pack-cache-lru-dynamic", "pack-cache-lru-static"] }
+gix = { version = "^0.53.0", path = "../gix", default-features = false, features = ["blob-diff", "revision", "mailmap", "excludes", "attributes", "worktree-mutation", "credentials", "interrupt"] }
+gix-pack-for-configuration-only = { package = "gix-pack", version = "^0.41.0", path = "../gix-pack", default-features = false, features = ["pack-cache-lru-dynamic", "pack-cache-lru-static", "generate", "streaming-input"] }
 gix-transport-configuration-only = { package = "gix-transport", version = "^0.35.0", path = "../gix-transport", default-features = false }
 gix-archive-for-configuration-only = { package = "gix-archive", version = "^0.3.0", path = "../gix-archive", optional = true, features = ["tar", "tar_gz"] }
 serde = { version = "1.0.114", optional = true, default-features = false, features = ["derive"] }

gitoxide-core/Cargo.toml

@@ -2,7 +2,7 @@ use std::borrow::Cow;
 use std::io;
 
 use anyhow::{anyhow, bail};
-use gix::{bstr::BStr, prelude::FindExt};
+use gix::bstr::BStr;
 
 use crate::{repository::PathsOrPatterns, OutputFormat};
 
@@ -49,7 +49,7 @@ pub fn query(
                     .metadata()
                     .ok()
                     .map(|m| m.is_dir());
-                let entry = cache.at_entry(path.as_slice(), is_dir, |oid, buf| repo.objects.find_blob(oid, buf))?;
+                let entry = cache.at_entry(path.as_slice(), is_dir)?;
                 let match_ = entry
                     .matching_exclude_pattern()
                     .and_then(|m| (show_ignore_patterns || !m.pattern.is_negative()).then_some(m));
@@ -68,7 +68,7 @@ pub fn query(
                 .index_entries_with_paths(&index)
                 .ok_or_else(|| anyhow!("Pathspec didn't yield any entry"))?
             {
-                let entry = cache.at_entry(path, Some(false), |oid, buf| repo.objects.find_blob(oid, buf))?;
+                let entry = cache.at_entry(path, Some(false))?;
                 let match_ = entry
                     .matching_exclude_pattern()
                     .and_then(|m| (show_ignore_patterns || !m.pattern.is_negative()).then_some(m));

gitoxide-core/src/repository/exclude.rs

@@ -43,7 +43,7 @@ document-features = { version = "0.2.0", optional = true }
 [dev-dependencies]
 gix-testtools = { path = "../tests/tools"}
 gix-odb = { path = "../gix-odb"}
-gix-worktree = { path = "../gix-worktree"}
+gix-worktree = { path = "../gix-worktree", default-features = false, features = ["attributes"]}
 gix-hash = { path = "../gix-hash"}
 gix-attributes = { path = "../gix-attributes"}
 gix-object = { path = "../gix-object"}

gix-archive/Cargo.toml

@@ -20,14 +20,14 @@ serde = ["dep:serde", "bstr/serde", "gix-glob/serde", "kstring/serde"]
 gix-path = { version = "^0.9.0", path = "../gix-path" }
 gix-quote = { version = "^0.4.7", path = "../gix-quote" }
 gix-glob = { version = "^0.11.0", path = "../gix-glob" }
+gix-trace = { version = "^0.1.3", path = "../gix-trace" }
 
 bstr = { version = "1.3.0", default-features = false, features = ["std", "unicode"]}
 smallvec = "1.10.0"
 kstring = "2.0.0"
 unicode-bom = "2.0.2"
 thiserror = "1.0.26"
 serde = { version = "1.0.114", optional = true, default-features = false, features = ["derive"]}
-log = "0.4.17"
 
 document-features = { version = "0.2.1", optional = true }
 

gix-attributes/Cargo.toml

@@ -124,7 +124,7 @@ impl Search {
 impl Pattern for Attributes {
     type Value = Value;
 
-    fn bytes_to_patterns(bytes: &[u8], source: &std::path::Path) -> Vec<pattern::Mapping<Self::Value>> {
+    fn bytes_to_patterns(bytes: &[u8], _source: &std::path::Path) -> Vec<pattern::Mapping<Self::Value>> {
         fn into_owned_assignments<'a>(
             attrs: impl Iterator<Item = Result<crate::AssignmentRef<'a>, crate::name::Error>>,
         ) -> Option<Assignments> {
@@ -138,8 +138,8 @@ impl Pattern for Attributes {
                 .collect::<Result<Assignments, _>>();
             match res {
                 Ok(res) => Some(res),
-                Err(err) => {
-                    log::warn!("{}", err);
+                Err(_err) => {
+                    gix_trace::warn!("{}", _err);
                     None
                 }
             }
@@ -148,8 +148,8 @@ impl Pattern for Attributes {
         crate::parse(bytes)
             .filter_map(|res| match res {
                 Ok(pattern) => Some(pattern),
-                Err(err) => {
-                    log::warn!("{}: {}", source.display(), err);
+                Err(_err) => {
+                    gix_trace::warn!("{}: {}", _source.display(), _err);
                     None
                 }
             })

gix-attributes/src/search/attributes.rs

@@ -24,7 +24,6 @@ gix-sec = { version = "^0.9.0", path = "../gix-sec" }
 gix-ref = { version = "^0.35.0", path = "../gix-ref" }
 gix-glob = { version = "^0.11.0", path = "../gix-glob" }
 
-log = "0.4.17"
 winnow = { version = "0.5.14", features = ["simd"] }
 memchr = "2"
 thiserror = "1.0.26"

gix-config/Cargo.toml

@@ -92,7 +92,7 @@ impl File<'static> {
                     Err(err) => {
                         let err = Error::Io { source: err, path };
                         if options.ignore_io_errors {
-                            log::warn!("ignoring: {err:#?}");
+                            gix_features::trace::warn!("ignoring: {err:#?}");
                             continue;
                         } else {
                             return Err(err);
@@ -104,7 +104,7 @@ impl File<'static> {
                 Ok(_) => {}
                 Err(err) => {
                     if options.ignore_io_errors {
-                        log::warn!(
+                        gix_features::trace::warn!(
                             "ignoring: {:#?}",
                             Error::Io {
                                 source: err,

gix-config/src/file/init/from_paths.rs

@@ -17,7 +17,7 @@ path = "config.rs"
 [dev-dependencies]
 gix-config = { path = ".."}
 gix-testtools = { path = "../../tests/tools"}
-gix = { path = "../../gix" }
+gix = { path = "../../gix", default-features = false }
 gix-ref = { path = "../../gix-ref" }
 gix-path = { path = "../../gix-path" }
 gix-sec = { path = "../../gix-sec" }