Thread (56 messages) 56 messages, 4 authors, 2020-07-15

Re: [PATCH 5/8] builtin/commit-graph.c: dereference tags in builtin

flat view

From: Derrick Stolee <hidden>
Date: 2020-05-05 12:01:33

On 5/4/2020 9:13 PM, Taylor Blau wrote:
quoted hunk ↗ jump to hunk
@@ -228,18 +240,25 @@ static int graph_write(int argc, const char **argv)
 		if (opts.stdin_commits) {
 			oidset_init(&commits, 0);
 			flags |= COMMIT_GRAPH_WRITE_CHECK_OIDS;
+			if (opts.progress)
+				progress = start_delayed_progress(
+					_("Analyzing commits from stdin"), 0);
The code functions as you intend and is an improvement. Similar to my
earlier suggestion to use something like "Collecting referenced commits"
for the --reachable case, perhaps this could be "Collecting commits from input"?

Thanks,
-Stolee
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help