[FFmpeg-devel] [PATCH] add colours to warnings and errors
Måns Rullgård
mans
Tue Apr 27 00:29:45 CEST 2010
James Darnley <james.darnley at gmail.com> writes:
> Right. See attached
>
> Index: libavutil/log.c
> ===================================================================
> --- libavutil/log.c (revision 22960)
> +++ libavutil/log.c (working copy)
> @@ -24,6 +24,10 @@
> * logging functions
> */
>
> +#ifdef _WIN32
> +#include <windows.h>
> +#include <string.h>
> +#endif
> #include <unistd.h>
> #include <stdlib.h>
> #include "avutil.h"
> @@ -34,18 +38,76 @@
> #endif
> int av_log_level = AV_LOG_INFO;
>
> +/* FIXME: On Windows isatty() returns true when ANSI color codes won't work.
> +Some hack to detect output to other terminals would be good, fixing the other
> +terminals would be better. One probable exception is when the user has
> +ANSI.SYS loaded but the Windows API should then still work. */
You still have a FIXME here. That's not acceptable.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list