Subject: Re: [PATCH 2/2] t7508-status: test all modes with color

From: Jakub Narebski <[email-masked]>
Date: Tue, 08 Dec 2009 12:10:58 +0100

Michael J Gruber wrote:

> +decrypt_color () {
> +       sed \
> +               -e 's/.\[1m/<WHITE>/g' \
> +               -e 's/.\[31m/<RED>/g' \
> +               -e 's/.\[32m/<GREEN>/g' \
> +               -e 's/.\[34m/<BLUE>/g' \
> +               -e 's/.\[m/<RESET>/g'
> +}

Shouldn't this be better in test-lib.sh, or some common lib 
(lib-color.sh or color-lib.sh; we are unfortunately a bit inconsistent
in naming here)?

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git