On 30/3/20 6:08 am, Carson Black wrote:

Version colour numbers are dulled in the non-verbose transaction summary when colours are enabled.

To prevent a regression, this patch also adds handling of strings with ANSI codes to string_length as to not break the transaction summary's output functions when colour codes are in the package name strings.

Signed-off-by: Carson Black <uhhadd@gmail.com> --- src/pacman/conf.c | 4 ++++ src/pacman/conf.h | 1 + src/pacman/util.c | 52 ++++++++++++++++++++++++++++++++++++----------- 3 files changed, 45 insertions(+), 12 deletions(-)

diff --git a/src/pacman/conf.c b/src/pacman/conf.c index f9de386f..d0b8e135 100644 --- a/src/pacman/conf.c +++ b/src/pacman/conf.c @@ -62,6 +62,7 @@ config_t *config = NULL; #define BOLDMAGENTA "\033[1;35m" #define BOLDCYAN "\033[1;36m" #define BOLDWHITE "\033[1;37m" +#define FAINTBLACK "\033[38;5;243m"

Just use the assigned colour name: GREY46

void enable_colors(int colors) { @@ -76,6 +77,7 @@ void enable_colors(int colors) colstr->meta = BOLDCYAN; colstr->warn = BOLDYELLOW; colstr->err = BOLDRED; + colstr->faint = FAINTBLACK; colstr->nocolor = NOCOLOR; } else { colstr->colon = ":: "; @@ -86,6 +88,7 @@ void enable_colors(int colors) colstr->meta = ""; colstr->warn = ""; colstr->err = ""; + colstr->faint = ""; colstr->nocolor = ""; } } @@ -120,6 +123,7 @@ config_t *config_new(void) newconfig->colstr.warn = ""; newconfig->colstr.err = ""; newconfig->colstr.nocolor = ""; + newconfig->colstr.faint = "";

Move up a line to be consistent with all other colour string orders.

return newconfig; } diff --git a/src/pacman/conf.h b/src/pacman/conf.h index d954e637..e1df24b7 100644 --- a/src/pacman/conf.h +++ b/src/pacman/conf.h @@ -31,6 +31,7 @@ typedef struct __colstr_t { const char *meta; const char *warn; const char *err; + const char *faint; const char *nocolor; } colstr_t;

diff --git a/src/pacman/util.c b/src/pacman/util.c index a3a85bb9..5042e0e3 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -415,12 +415,40 @@ static size_t string_length(const char *s) if(!s || s[0] == '\0') { return 0; } - /* len goes from # bytes -> # chars -> # cols */ - len = strlen(s) + 1; - wcstr = calloc(len, sizeof(wchar_t)); - len = mbstowcs(wcstr, s, len); - len = wcswidth(wcstr, len); - free(wcstr); + if(strstr(s, "\033")) { + const char* source = s; + char* replaced = malloc(sizeof(char)*strlen(s));

Add spaces around "*"

+ int mode = 0; + int iter = 0; + for(char character = *source; character != '\0'; character = *++source) { + if(mode == 0) { + if(character == '\033') { + mode = 1; + } else { + replaced[iter] = character; + iter++; + } + } else if(mode == 1) { + if (character == 'm') { + mode = 0; + } + } + }

Lot of unneeded stuff going on there. Replace that entire block with: int iter = 0; for(; *s; s++) { if(*s == '\033') { while(*s != 'm') { s++; } } else { replaced[iter] = *s; iter++; } }

+ replaced[iter] = '\0'; + len = iter; + wcstr = calloc(len, sizeof(wchar_t)); + len = mbstowcs(wcstr, replaced, len); + len = wcswidth(wcstr, len); + free(wcstr); + free(replaced); + } else { + /* len goes from # bytes -> # chars -> # cols */ + len = strlen(s) + 1; + wcstr = calloc(len, sizeof(wchar_t)); + len = mbstowcs(wcstr, s, len); + len = wcswidth(wcstr, len); + free(wcstr); + }

return len; } @@ -905,14 +933,14 @@ static void _display_targets(alpm_list_t *targets, int verbose) }

if(target->install) { - pm_asprintf(&str, "%s-%s", alpm_pkg_get_name(target->install), - alpm_pkg_get_version(target->install)); + pm_asprintf(&str, "%s%s-%s%s", alpm_pkg_get_name(target->install), config->colstr.faint, + alpm_pkg_get_version(target->install), config->colstr.nocolor); } else if(isize == 0) { - pm_asprintf(&str, "%s-%s", alpm_pkg_get_name(target->remove), - alpm_pkg_get_version(target->remove)); + pm_asprintf(&str, "%s%s-%s%s", alpm_pkg_get_name(target->remove), config->colstr.faint, + alpm_pkg_get_version(target->remove), config->colstr.nocolor); } else { - pm_asprintf(&str, "%s-%s [%s]", alpm_pkg_get_name(target->remove), - alpm_pkg_get_version(target->remove), _("removal")); + pm_asprintf(&str, "%s%s-%s %s[%s]%s", alpm_pkg_get_name(target->remove), config->colstr.faint, + alpm_pkg_get_version(target->remove), config->colstr.nocolor, _("removal"), config->colstr.nocolor); } names = alpm_list_add(names, str); }