On Tue, Aug 21, 2018 at 3:13 AM Jeff King [off-list ref] wrote:
I _think_ they should work together OK without further modification.
Once upon a time the caller had to say "don't show if we're past N%
after M seconds", but I think with the current code we'd just show it if
we're not completely finished after 2 seconds.
So it really should just be a simple:
progress = start_delayed_progress("Hashing packfile", 0);
That said, counting bytes would probably be ugly (just because the
numbers get really big). We'd probably prefer to show a throughput or
something.
Or just an ascii throbber. I think the important thing is communicate
"I'm still doing something, not hanging up". "Hashing packfile"
satisfies the "something" part, the throbber the "hanging".
--
Duy