[MPlayer-cvslog] r26174 - trunk/configure

[Diego Biurrun]

On Sat, Mar 08, 2008 at 11:43:12AM +0200, Ivan Kalvachev wrote:
> On Thu, Mar 6, 2008 at 2:58 AM, Diego Biurrun <diego at biurrun.de> wrote:
> > On Thu, Mar 06, 2008 at 01:14:55AM +0200, Uoti Urpala wrote:
> >  > On Thu, 2008-03-06 at 00:04 +0100, diego wrote:
> >  > > Do not enable grayscale decoding in FFmpeg, it slows down the
> >  > > default (color) case.
> >  >
> >  > > -/* Support for grayscale encoding/decoding in FFmpeg (makes color slower) */
> >  > > -#define ENABLE_GRAY 1
> >  >
> >  > If you change this you should clean up references elsewhere (at least
> >  > options and man page entries). See also
> >  > http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/2007-August/030763.html
> >
> >  Yes.  What would this entail exactly?  Just DOCS/man/en/mplayer.1,
> >  libmpcodecs/ve_lavc.c and libmpcodecs/vd_ffmpeg.c like in attached
> >  patch or did I miss something?
> 
> Why is the disable unconditional and not configure option?

Feel free to implement it if you wish to have it.

> The gray alone gives same speedup as lowres=3 (the maximum working
> level that results in 16 times smaller image and noticeable
> artifacts).

Only when it is not broken, but unfortunately it is broken.

Diego