[FFmpeg-cvslog] r9846 - in trunk: configure libavcodec/mpegvideo.c libavcodec/mpegvideo_common.h

[Diego Biurrun]

On Wed, Aug 01, 2007 at 02:58:22PM +0200, Michael Niedermayer wrote:
> 
> On Wed, Aug 01, 2007 at 08:26:29AM +0100, M?ns Rullg?rd wrote:
> > michael <subversion at mplayerhq.hu> writes:
> > 
> > > Log:
> > > make grayscale only decoding checks in inner loops compiletime killable
> > >
> > > --- trunk/configure	(original)
> > > +++ trunk/configure	Wed Aug  1 02:39:39 2007
> > > @@ -615,6 +616,7 @@ CONFIG_LIST='
> > >      v4l2
> > >      x11grab
> > >      zlib
> > > +    gray
> > >  '
> > 
> > Please maintain alphabetical order.  IIRC this is even mentioned in
> > some documentation.
> 
> yes it is in the docs, but this list is not in alphabetical order:
> CONFIG_LIST='
>     encoders
>     decoders
>     parsers
>     bsfs
>     muxers
>     demuxers
>     audio_beos
> ...
>     zlib
> 
> ive moved gray so as to keep the list closer to alphabetically ordered

Thanks for pointing this out, fixed.

> > > @@ -860,6 +862,7 @@ network="yes"
> > >  optimize="yes"
> > >  protocols="yes"
> > >  vhook="default"
> > > +gray="no"
> > 
> > Not necessary.  The default is no.
> 
> fixed
> 
> note, gray is not the only ="no" in there, gpl, shared, memalign_hack are
> too

Thanks for pointing this out, fixed.

Diego