[FFmpeg-devel] [PATCH] Add Win32 check for FFMPEG_FORCE_NOCOLOR

[Michael Niedermayer]

On Fri, Dec 31, 2010 at 01:35:55PM -0700, Daniel Verkamp wrote:
> Hi,
> 
> Currently the Win32 path of libavutil/log.c only checks the NO_COLOR
> environment variable; a check for the recently-added
> FFMPEG_FORCE_NOCOLOR is missing.
> 
> This fixes issue 2461.
> 
> Thanks,
> -- Daniel Verkamp

> Index: libavutil/log.c
> ===================================================================
> --- libavutil/log.c	(revision 26178)
> +++ libavutil/log.c	(working copy)
> @@ -55,7 +55,7 @@
>  #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("NO_COLOR") && !getenv("FFMPEG_FORCE_NOCOLOR");

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110101/ff6d1191/attachment.pgp>