On Wed, Sep 9, 2015 at 8:27 PM, Timothy Gu <timothyg...@gmail.com> wrote:
> On Wed, Sep 9, 2015 at 11:15 AM Ganesh Ajjanagadde <gajjanaga...@gmail.com>
> wrote:
>>
>> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com>
>> ---
>> configure | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index 056feea..4842b52 100755
>> --- a/configure
>> +++ b/configure
>> @@ -421,6 +421,7 @@ if test -t 1 && which tput >/dev/null; then
>> if test -n "$ncolors" && test $ncolors -ge 8; then
>> bold_color=$(tput bold)
>> warn_color=$(tput setaf 3)
>> + error_color=$(tput setaf 1)
>> reset_color=$(tput sgr0)
>> fi
>> fi
>> @@ -437,7 +438,7 @@ log_file(){
>>
>> echolog(){
>> log "$@"
>> - echo "$@"
>> + echo "$error_color$bold_color$@$reset_color"
>> }
>
>
> You can inline this function in die() which is the only place this is used.
> The function name `echolog` doesn't do anything to indicate that what's
> echoed is an error.
Thanks; updated patch.
>
> Timothy
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
