Merge pull request #6319 from libgit2/ethomson/progress_32bit CLI: progress updates
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67
diff --git a/src/cli/progress.c b/src/cli/progress.c
index 39b5953..ba52655 100644
--- a/src/cli/progress.c
+++ b/src/cli/progress.c
@@ -12,7 +12,13 @@
#include "progress.h"
#include "error.h"
-#define PROGRESS_UPDATE_TIME 0.05
+/*
+ * Show updates to the percentage and number of objects received
+ * separately from the throughput to give an accurate progress while
+ * avoiding too much noise on the screen.
+ */
+#define PROGRESS_UPDATE_TIME 0.10
+#define THROUGHPUT_UPDATE_TIME 1.00
#define is_nl(c) ((c) == '\r' || (c) == '\n')
@@ -200,11 +206,20 @@ static int fetch_receiving(
else
now = git__timer();
- recv_len = (double)stats->received_bytes;
+ if (progress->throughput_update &&
+ now - progress->throughput_update < THROUGHPUT_UPDATE_TIME) {
+ elapsed = progress->throughput_update -
+ progress->action_start;
+ recv_len = progress->throughput_bytes;
+ } else {
+ elapsed = now - progress->action_start;
+ recv_len = (double)stats->received_bytes;
+
+ progress->throughput_update = now;
+ progress->throughput_bytes = recv_len;
+ }
- elapsed = now - progress->action_start;
rate = elapsed ? recv_len / elapsed : 0;
- done = (stats->received_objects == stats->total_objects);
while (recv_len > 1024 && recv_units[recv_unit_idx+1]) {
recv_len /= 1024;
@@ -295,7 +310,7 @@ void cli_progress_checkout(
}
progress_printf(progress, false,
- "Checking out files: %3d%% (%lu/%lu)%s\r",
+ "Checking out files: %3d%% (%" PRIuZ "/%" PRIuZ ")%s\r",
percent(completed_steps, total_steps),
completed_steps, total_steps,
done ? ", done." : "");
diff --git a/src/cli/progress.h b/src/cli/progress.h
index fab2e97..7a445ec 100644
--- a/src/cli/progress.h
+++ b/src/cli/progress.h
@@ -40,6 +40,10 @@ typedef struct {
git_str sideband;
git_str onscreen;
git_str deferred;
+
+ /* Last update about throughput */
+ double throughput_update;
+ double throughput_bytes;
} cli_progress;
#define CLI_PROGRESS_INIT { 0 }