[FFmpeg-cvslog] r9307 - in trunk/libavcodec: dsputil.c mpegvideo.c

[Diego Biurrun]

On Fri, Jun 15, 2007 at 12:56:10PM +0200, Reimar Doeffinger wrote:
> On Fri, Jun 15, 2007 at 10:17:09AM +0200, Diego Biurrun wrote:
> > On Fri, Jun 15, 2007 at 10:06:55AM +0200, Reimar Doeffinger wrote:
> > > On Thu, Jun 14, 2007 at 10:44:41AM +0200, diego wrote:
> > > > 
> > > > Log:
> > > > Simplify init preprocessor statements.
> > > > patch by Albert Lee, trisk+xine acm.jhu edu
> > > 
> > > As discussed this has a sideeffect and IMO also removes a feature (using
> > > accelerated functions from different "areas"), so I'd request this to be
> > > reverted unless there is a really, really good reason to do it like
> > > this (and even then the log message is not okay).
> > 
> > AFAICT all of these options except for mlib are mutually exclusive,
> > right?  So the right solution IMO is to take just mlib out of the #if
> > #elif construction.
> 
> I do not know, and what is the advantage that would justify trying to
> find out? Just to get rid of 12 #endifs?

That's not a bad reason IMO.  It also represents the semantics a tad
better for my taste.

> If it bothers you as much, IMO just switch to the "if (ENABLED_*)"
> construct used elsewhere, then you can even put it all in one line.

I'm fine with that.

Diego