[MPlayer-cvslog] r26174 - trunk/configure

[Ivan Kalvachev]

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?

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

Also I had tried to optimize mplayer with -fprofile-arcs then
-fbranch-probabilities . The result one is still 4% faster than
mplayer compiled only with disabled gray.

(The difference may be also because gcc4.1.2 produces faster binaries
than gcc421, I will probably try to optimize another one, soon. These
days ccache is not so efficient with all these cosmetics and headers
messing).