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

Reimar Doeffinger Reimar.Doeffinger
Sun Jun 17 18:23:11 CEST 2007


Hello,
On Sun, Jun 17, 2007 at 04:20:03PM +0200, Diego Biurrun wrote:
> On Fri, Jun 15, 2007 at 12:56:10PM +0200, Reimar Doeffinger wrote:
> > 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.

Well, after you changed the semantics with that patch of course :-P
IMHO it is not desireable to allow only one set of optimization function
per build, though I am at loss for a _really_ good argument right now
(unless we might want to split things into dsputil_init_mmx,
dsputil_sse, dsputil_3dnow somewhen. Did not investigate if that would
simplify things).

Greetings,
Reimar Doeffinger




More information about the ffmpeg-cvslog mailing list