From c35f713e751bf74bd7b75e683a7decc549eab60f Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Wed, 31 Jan 2007 01:37:41 +0000 Subject: * Refactored some functions to clean up variable declaration. * Reduced magic number usage by fill_progress function (new). * Some switch indent fixing. * Remove use of log10 call. --- src/pacman/trans.c | 193 ++++++++++++++++++++++++++++------------------------- src/pacman/trans.h | 4 +- 2 files changed, 104 insertions(+), 93 deletions(-) diff --git a/src/pacman/trans.c b/src/pacman/trans.c index 6e0c8428..f2f7113c 100644 --- a/src/pacman/trans.c +++ b/src/pacman/trans.c @@ -26,7 +26,6 @@ #include #include #include -#include #include #include @@ -42,13 +41,79 @@ extern config_t *config; static int prevpercent=0; /* for less progressbar output */ +/* refactored function from cb_trans_evt */ +static void retrieve_local(void *data1, void *data2) +{ + const unsigned int maxcols = getcols(); + char out[PATH_MAX]; + unsigned int i; + + MSG(NL, " %s [", (char*)data1); + STRNCPY(out, (char*)data2, maxcols-42); + MSG(CL, "%s", out); + for(i = strlen(out); i < maxcols-43; i++) { + MSG(CL, " "); + } + fputs(_("] 100% LOCAL "), stdout); +} + +/* refactored from cb_trans_progress */ +/* TODO with a little more work, we may be able to incorporate this + * into the download progress bar as well. */ +static void fill_progress(const int percent, const int proglen) +{ + const unsigned short chomp = alpm_option_get_chomp(); + const unsigned int hashlen = proglen - 8; + const unsigned int hash = percent * hashlen / 100; + unsigned int lasthash = 0, mouth = 0; + unsigned int i; + + printf(" ["); + for(i = hashlen; i > 1; --i) { + /* if special progress bar enabled */ + if(chomp) { + if(i > hashlen - hash) { + printf("-"); + } else if(i == hashlen - hash) { + if(lasthash == hash) { + if(mouth) { + printf("\033[1;33mC\033[m"); + } else { + printf("\033[1;33mc\033[m"); + } + } else { + lasthash = hash; + mouth = mouth == 1 ? 0 : 1; + if(mouth) { + printf("\033[1;33mC\033[m"); + } else { + printf("\033[1;33mc\033[m"); + } + } + } else if(i%3 == 0) { + printf("\033[0;37mo\033[m"); + } else { + printf("\033[0;37m \033[m"); + } + } /* else regular progress bar */ + else if(i > hashlen - hash) { + printf("#"); + } else { + printf("-"); + } + } + printf("] %3d%%\r", percent); + + if(percent == 100) { + printf("\n"); + } +} + /* Callback to handle transaction events */ void cb_trans_evt(pmtransevt_t event, void *data1, void *data2) { char str[LOG_STR_LEN] = ""; - char out[PATH_MAX]; - int i; switch(event) { case PM_TRANS_EVT_CHECKDEPS_START: @@ -155,14 +220,7 @@ void cb_trans_evt(pmtransevt_t event, void *data1, void *data2) fflush(stdout); break; case PM_TRANS_EVT_RETRIEVE_LOCAL: - MSG(NL, " %s [", (char*)data1); - unsigned int maxcols = getcols(); - STRNCPY(out, (char*)data2, maxcols-42); - MSG(CL, "%s", out); - for(i = strlen(out); i < maxcols-43; i++) { - MSG(CL, " "); - } - fputs(_("] 100% LOCAL "), stdout); + retrieve_local(data1, data2); break; } } @@ -286,14 +344,12 @@ void cb_trans_conv(pmtransconv_t event, void *data1, void *data2, } } -void cb_trans_progress(pmtransprog_t event, char *pkgname, int percent, - int howmany, int remain) +void cb_trans_progress(pmtransprog_t event, char *pkgname, const int percent, + const int howmany, const int remain) { - static int lasthash = 0, mouth = 0; - int i, hash; - long chomp = 0; - unsigned int maxcols = getcols(); - unsigned int maxpkglen, progresslen = maxcols - 57; + /* size of line to allocate for text printing (e.g. not progressbar) */ + const int infolen = 50; + int i, digits, textlen, pkglen, proglen; char *ptr = NULL; if(config->noprogressbar) { @@ -306,103 +362,58 @@ void cb_trans_progress(pmtransprog_t event, char *pkgname, int percent, set_output_padding(0); /* shut it off again */ } - if (!pkgname) - return; - if (percent > 100) - return; - if(percent == prevpercent) + /* if no pkgname, percent is too high or unchanged, then return */ + if (!pkgname || percent > 100 || percent == prevpercent) { return; + } prevpercent=percent; switch (event) { case PM_TRANS_PROGRESS_ADD_START: ptr = _("installing"); break; - case PM_TRANS_PROGRESS_UPGRADE_START: ptr = _("upgrading"); break; - case PM_TRANS_PROGRESS_REMOVE_START: ptr = _("removing"); break; - case PM_TRANS_PROGRESS_CONFLICTS_START: ptr = _("checking for file conflicts"); break; } - hash=percent*progresslen/100; - // if the package name is too long, then slice the ending - maxpkglen=46-strlen(ptr)-(3+2*(int)log10(howmany)); - if(strlen(pkgname)>maxpkglen) - pkgname[maxpkglen]='\0'; + /* find # of digits in package counts to scale output */ + digits = 1; + i = howmany; + while((i /= 10)) { + ++digits; + } + + /* determine room left for non-digits text [not ( 1/12) part] */ + textlen = infolen - 3 - (2 * digits); + /* room left for package name */ + pkglen = textlen - strlen(ptr) - 1; switch (event) { - case PM_TRANS_PROGRESS_ADD_START: - case PM_TRANS_PROGRESS_UPGRADE_START: - case PM_TRANS_PROGRESS_REMOVE_START: - putchar('('); - for(i=0;i<(int)log10(howmany)-(int)log10(remain);i++) - putchar(' '); - printf("%d/%d) %s %s ", remain, howmany, ptr, pkgname); - if (strlen(pkgname)0; i--) - putchar(' '); - break; - - case PM_TRANS_PROGRESS_CONFLICTS_START: - printf("%s (", ptr); - for(i=0;i<(int)log10(howmany)-(int)log10(remain);i++) - putchar(' '); - printf("%d/%d) ", remain, howmany); - for (i=maxpkglen; i>0; i--) - putchar(' '); - break; - } + case PM_TRANS_PROGRESS_ADD_START: + case PM_TRANS_PROGRESS_UPGRADE_START: + case PM_TRANS_PROGRESS_REMOVE_START: + /* TODO clean up so digits and pkglen aren't passed twice */ + printf("(%*d/%*d) %s %-*.*s", digits, remain, digits, howmany, + ptr, pkglen, pkglen, pkgname); + break; - chomp = alpm_option_get_chomp(); + case PM_TRANS_PROGRESS_CONFLICTS_START: + printf("(%*d/%*d) %-*s", digits, remain, digits, howmany, + textlen, ptr); + break; - /* hide the cursor, prevent flicker during fancy graphics - printf("\033[?25l\033[?1c["); - */ - printf("["); - for(i = progresslen; i > 0; --i) { - if(chomp) { - if(i > progresslen - hash) { - printf("-"); - } else if(i == progresslen - hash) { - if(lasthash == hash) { - if(mouth) { - printf("\033[1;33mC\033[m"); - } else { - printf("\033[1;33mc\033[m"); - } - } else { - lasthash = hash; - mouth = mouth == 1 ? 0 : 1; - if(mouth) { - printf("\033[1;33mC\033[m"); - } else { - printf("\033[1;33mc\033[m"); - } - } - } else if(i%3 == 0) { - printf("\033[0;37mo\033[m"); - } else { - printf("\033[0;37m \033[m"); - } - } else if(i > progresslen - hash) { - printf("#"); - } else { - printf("-"); - } } - printf("] %3d%%\r", percent); - if(percent == 100) { - printf("\n"); - } + /* call refactored fill progress function */ + proglen = getcols() - infolen; + fill_progress(percent, proglen); } /* vim: set ts=2 sw=2 noet: */ diff --git a/src/pacman/trans.h b/src/pacman/trans.h index 6eb78358..63ff4714 100644 --- a/src/pacman/trans.h +++ b/src/pacman/trans.h @@ -29,8 +29,8 @@ void cb_trans_conv(pmtransconv_t event, void *data1, void *data2, void *data3, int *response); /* callback to handle display of the progress bar for transactions */ -void cb_trans_progress(pmtransprog_t event, char *pkgname, int percent, - int howmany, int remain); +void cb_trans_progress(pmtransprog_t event, char *pkgname, const int percent, + const int howmany, const int remain); #endif /* _PM_TRANS_H */ -- cgit v1.2.3-24-g4f1b