[FFmpeg-devel] Allow to force colored output

Stefano Sabatini stefano.sabatini-lala
Sun Nov 14 14:23:29 CET 2010


On date Sunday 2010-11-14 14:05:03 +0100, Etienne Buira encoded:
> Hi.
> 
> Allow to force colored output even if stderr doesn't look to sanely
> interpret it.
> 
> Regards.

> Index: libavutil/log.c
> ===================================================================
> --- libavutil/log.c	(revision 25751)
> +++ libavutil/log.c	(working copy)
> @@ -66,6 +66,7 @@
>  #else
>          use_color= 0;
>  #endif
> +        use_color = use_color || getenv("FORCE_COLOR");
>      }
>  
>      if(use_color){
> Index: doc/fftools-common-opts.texi
> ===================================================================
> --- doc/fftools-common-opts.texi	(revision 25751)
> +++ doc/fftools-common-opts.texi	(working copy)
> @@ -84,6 +84,7 @@
>  
>  By default the program logs to stderr, if coloring is supported by the
>  terminal, colors are used to mark errors and warnings. Log coloring
> -can be disabled setting the environment variable @env{NO_COLOR}.
> +can be disabled setting the environment variable @env{NO_COLOR}, or
> +can be forced setting the environment variable @env{FORCE_COLOR}.

We should provide a namespace even in this case, I suggest:
FFMPEG_NO_COLOR
FFMPEG_FORCE_COLOR

Also can you say why do you need this?
-- 
FFmpeg = Fascinating Fiendish Mean Practical Excellent Gnome



More information about the ffmpeg-devel mailing list