commit-graph.c: simplify 'fill_oids_from_commits'

In the previous handful of commits, both 'git commit-graph write
--reachable' and '--stdin-commits' learned to peel tags down to the
commits which they refer to before passing them into the commit-graph
internals.

This makes the call to 'lookup_commit_reference_gently()' inside of
'fill_oids_from_commits()' a noop, since all OIDs are commits by that
point.

As such, remove the call entirely, as well as the progress meter, which
has been split and moved out to the callers in the aforementioned
earlier commits.

Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/commit-graph.c b/commit-graph.c
index 6098ecd5752ea46621517d8e3dd9fa387b495517..aed03f4b2f7ec21807414459631a301d5055727f 100644 (file)
--- a/commit-graph.c
+++ b/commit-graph.c
@@ -1412,46 +1412,19 @@ static int fill_oids_from_packs(struct write_commit_graph_context *ctx,
 static int fill_oids_from_commits(struct write_commit_graph_context *ctx,
                                  struct oidset *commits)
 {
-       uint32_t i = 0;
-       struct strbuf progress_title = STRBUF_INIT;
        struct oidset_iter iter;
        struct object_id *oid;
 
        if (!oidset_size(commits))
                return 0;
 
-       if (ctx->report_progress) {
-               strbuf_addf(&progress_title,
-                           Q_("Finding commits for commit graph from %d ref",
-                              "Finding commits for commit graph from %d refs",
-                              oidset_size(commits)),
-                           oidset_size(commits));
-               ctx->progress = start_delayed_progress(
-                                       progress_title.buf,
-                                       oidset_size(commits));
-       }
-
        oidset_iter_init(commits, &iter);
        while ((oid = oidset_iter_next(&iter))) {
-               struct commit *result;
-
-               display_progress(ctx->progress, ++i);
-
-               result = lookup_commit_reference_gently(ctx->r, oid, 1);
-               if (result) {
-                       ALLOC_GROW(ctx->oids.list, ctx->oids.nr + 1, ctx->oids.alloc);
-                       oidcpy(&ctx->oids.list[ctx->oids.nr], &(result->object.oid));
-                       ctx->oids.nr++;
-               } else if (ctx->check_oids) {
-                       error(_("invalid commit object id: %s"),
-                             oid_to_hex(oid));
-                       return -1;
-               }
+               ALLOC_GROW(ctx->oids.list, ctx->oids.nr + 1, ctx->oids.alloc);
+               oidcpy(&ctx->oids.list[ctx->oids.nr], oid);
+               ctx->oids.nr++;
        }
 
-       stop_progress(&ctx->progress);
-       strbuf_release(&progress_title);
-
        return 0;
 }