Make path optional

Author: cruessler

gitoxide-core/src/repository/log.rs

@@ -2,9 +2,32 @@ use gix::bstr::{BStr, BString, ByteSlice};
 use gix::prelude::FindExt;
 use gix::ObjectId;
 
-pub fn log(mut repo: gix::Repository, out: &mut dyn std::io::Write, path: BString) -> anyhow::Result<()> {
+pub fn log(mut repo: gix::Repository, out: &mut dyn std::io::Write, path: Option<BString>) -> anyhow::Result<()> {
     repo.object_cache_size_if_unset(repo.compute_object_cache_size_for_tree_diffs(&**repo.index_or_empty()?));
 
+    if let Some(path) = path {
+        log_file(repo, out, path)
+    } else {
+        log_all(repo, out)
+    }
+}
+
+fn log_all(repo: gix::Repository, out: &mut dyn std::io::Write) -> Result<(), anyhow::Error> {
+    let head = repo.head()?.peel_to_commit_in_place()?;
+    let mut topo =
+        gix::traverse::commit::topo::Builder::from_iters(&repo.objects, [head.id], None::<Vec<gix::ObjectId>>)
+            .build()?;
+
+    while let Some(info) = topo.next() {
+        let info = info?;
+
+        write_info(&repo, &mut *out, &info)?;
+    }
+
+    Ok(())
+}
+
+fn log_file(repo: gix::Repository, out: &mut dyn std::io::Write, path: BString) -> anyhow::Result<()> {
     let head = repo.head()?.peel_to_commit_in_place()?;
     let mut topo =
         gix::traverse::commit::topo::Builder::from_iters(&repo.objects, [head.id], None::<Vec<gix::ObjectId>>)

src/plumbing/options/mod.rs

@@ -418,7 +418,7 @@ pub mod log {
     pub struct Platform {
         /// The git path specification to show a log for.
         #[clap(value_parser = crate::shared::AsBString)]
-        pub pathspec: BString,
+        pub pathspec: Option<BString>,
     }
 }