[FFmpeg-devel] Allow to force colored output

Stefano Sabatini stefano.sabatini-lala
Sun Nov 14 15:47:06 CET 2010


On date Sunday 2010-11-14 15:02:58 +0100, Etienne Buira encoded:
> On Sun, Nov 14, 2010 at 02:44:02PM +0100, Etienne Buira wrote:
> > On Sun, Nov 14, 2010 at 02:23:29PM +0100, Stefano Sabatini wrote:
[...]
> > > We should provide a namespace even in this case, I suggest:
> > > FFMPEG_NO_COLOR
> > > FFMPEG_FORCE_COLOR
> > 
> > Done, but might break for people who use NO_COLOR yet.
> > 
> > > Also can you say why do you need this?
> > 
> > My usecase is to save ffmpeg's output (ffmpeg ... > log 2>&1), but still
> > having nice colors.
> 
> Sorry for the noise, this one should actually not screw up things for
> win32.

> Index: libavutil/log.c
> ===================================================================
> --- libavutil/log.c	(revision 25751)
> +++ libavutil/log.c	(working copy)
> @@ -55,16 +55,16 @@
>  #if defined(_WIN32) && !defined(__MINGW32CE__)
>          CONSOLE_SCREEN_BUFFER_INFO con_info;
>          con = GetStdHandle(STD_ERROR_HANDLE);
> -        use_color = (con != INVALID_HANDLE_VALUE) && !getenv("NO_COLOR");
> +        use_color = (con != INVALID_HANDLE_VALUE) && (!getenv("FFMPEG_NO_COLOR") || getenv("FFMPEG_FORCE_COLOR"));
>          if (use_color) {
>              GetConsoleScreenBufferInfo(con, &con_info);
>              attr_orig  = con_info.wAttributes;
>              background = attr_orig & 0xF0;
>          }
>  #elif HAVE_ISATTY
> -        use_color= getenv("TERM") && !getenv("NO_COLOR") && isatty(2);
> +        use_color= (getenv("TERM") && !getenv("FFMPEG_NO_COLOR") && isatty(2)) || getenv("FFMPEG_FORCE_COLOR");
>  #else
> -        use_color= 0;
> +        use_color= !!getenv("FFMPEG_FORCE_COLOR");
>  #endif
>      }

Don't mix separate changes, simply call the new variable
FFMPEG_FORCE_COLOR, the change NO_COLOR -> FFMPEG_NO_COLOR belongs to
a different patch.
-- 
FFmpeg = Friendly and Fantastic Mean Pure Eretic Gigant



More information about the ffmpeg-devel mailing list