summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorXavier Chantry <shiningxc@gmail.com>2009-12-16 02:16:00 +0100
committerDan McGee <dan@archlinux.org>2010-03-15 00:39:26 +0100
commite5dce888c09976f1ef4664613c742d0d015e534e (patch)
tree2e22c2160d19ab6ae59257f2967bf5cdcea72def
parent087be2f1fbae2d7168732c7004d8963a3d9d21d8 (diff)
downloadpacman-e5dce888c09976f1ef4664613c742d0d015e534e.tar.gz
pacman-e5dce888c09976f1ef4664613c742d0d015e534e.tar.xz
callback.c : less magic progress bars
1 - Explain magic numbers 2 - There was a weird off by 1 mess in the progress bar. The code supposedly shared the width between 50 chars for text (textlen) and the rest for the progress bar (proglen = getcols() - textlen). But the code actually used textlen + 1 for the text and proglen - 1 for the progress bar (with haslen=1, the progress bar was actually empty), which was a bit confusing so I changed it. Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r--src/pacman/callback.c23
1 files changed, 13 insertions, 10 deletions
diff --git a/src/pacman/callback.c b/src/pacman/callback.c
index 858bfdf4..aa1d2633 100644
--- a/src/pacman/callback.c
+++ b/src/pacman/callback.c
@@ -90,20 +90,20 @@ static float get_update_timediff(int first_call)
static void fill_progress(const int bar_percent, const int disp_percent,
const int proglen)
{
- const unsigned int hashlen = proglen - 8;
- const unsigned int hash = bar_percent * hashlen / 100;
- static unsigned int lasthash = 0, mouth = 0;
- unsigned int i;
+ /* 8 = 1 space + 1 [ + 1 ] + 5 for percent */
+ const int hashlen = proglen - 8;
+ const int hash = bar_percent * hashlen / 100;
+ static int lasthash = 0, mouth = 0;
+ int i;
if(bar_percent == 0) {
lasthash = 0;
mouth = 0;
}
- /* magic numbers, how I loathe thee */
- if(proglen > 8) {
+ if(hashlen > 0) {
printf(" [");
- for(i = hashlen; i > 1; --i) {
+ for(i = hashlen; i > 0; --i) {
/* if special progress bar enabled */
if(config->chomp) {
if(i > hashlen - hash) {
@@ -139,7 +139,8 @@ static void fill_progress(const int bar_percent, const int disp_percent,
printf("]");
}
/* print display percent after progress bar */
- if(proglen > 5) {
+ /* 5 = 1 space + 3 digits + 1 % */
+ if(proglen >= 5) {
printf(" %3d%%", disp_percent);
}
@@ -374,7 +375,7 @@ void cb_trans_progress(pmtransprog_t event, const char *pkgname, int percent,
++digits;
}
/* determine room left for non-digits text [not ( 1/12) part] */
- textlen = infolen - 3 - (2 * digits);
+ textlen = infolen - 3 /* (/) */ - (2 * digits) - 1 /* space */;
/* In order to deal with characters from all locales, we have to worry
* about wide characters and their column widths. A lot of stuff is
@@ -449,7 +450,8 @@ void cb_dl_total(off_t total)
void cb_dl_progress(const char *filename, off_t file_xfered, off_t file_total)
{
const int infolen = 50;
- const int filenamelen = infolen - 27;
+ /* explanation of magic 28 number at the end */
+ const int filenamelen = infolen - 28;
char *fname, *p;
/* used for wide character width determination and printing */
int len, wclen, wcwid, padwid;
@@ -611,6 +613,7 @@ void cb_dl_progress(const char *filename, off_t file_xfered, off_t file_total)
}
}
+ /* 1 space + filenamelen + 1 space + 7 for size + 1 + 7 for rate + 2 for /s + 1 space + 8 for eta */
printf(" %ls%-*s %6.1f%c %#6.1f%c/s %02u:%02u:%02u", wcfname,
padwid, "", f_xfered, xfered_size,
rate, rate_size, eta_h, eta_m, eta_s);