[FFmpeg-devel] [PATCH] add colours to warnings and errors

François Revol revol
Thu Apr 22 18:36:18 CEST 2010


How about using termcap/terminfo/* instead of hardcoding ANSI codes ?

> >>> The current code adds a black background.
> >>>
> >>> As long as we're bikeshedding, I want it not to change the
> >>> background, the entire line be in bold, and only the "[%s @ %p]" 
> > > > be
> >>> colorized if present.
> >>
> >> blink, slow or rapid? :)
> > My idea:
> >
> > AV_LOG_FATAL: red blinking
> > AV_LOG_ERROR: red
> > AV_LOG_WARN: yellow
> >
> > Regarding changing the background, we yesterday discussed in
> > #ffmpeg-devel that it's maybe the best idea to implement some high
> > contrast algorithm.

Note sometimes the background is actually a (semi)transparent color, 
and forcing a background color fills the whole window with an opaque 
version which really sux visually (vi and others do it).

Btw, there are other options, like underline (4) or video inverse (7).

> It's not generally possible to query the background colour.

Indeed, doesn't seem to be available in termcap.

> > Another option would be one being able to set color with 
> > configure...
> >
> > Sth. like that?
> > ../configure --warn-fg-color=2 --warn-bg-color=3 --err-fg-color=4
> > --err-bg-color=5 --fatal-fg-color=6 --fatal-bg-color=7
> 
> No, NO, *NO*!!!

Wow, first time in a while I agree with M?ns :D

It's odd noone came up with some kind of standardization for that...
Maybe we could snoop on the $DIRCOLORS, or .vimrc or .emacs/init.el :D

Fran?ois.



More information about the ffmpeg-devel mailing list