[Ffmpeg-devel] [PATCH] Snow mmx+sse2 asm optimizations

Michael Niedermayer michaelni
Thu Mar 16 10:45:00 CET 2006


Hi

On Tue, Mar 14, 2006 at 01:44:03PM +0200, Ivan Kalvachev wrote:
> 2006/3/14, Robert Edele <yartrebo at earthlink.net>:
> > On Mon, 2006-03-13 at 02:52 +0100, Michael Niedermayer wrote:
> > > ok, first patch looks mostly ok, iam not particulary happy about the
> > > inclusion of snow.h in dsputil.h but i dont really care
> > > as dsputil.h was never supposed to be a public header, so whoever
> > > came up with that idea can fix the snow.h inclusion (installing snow.h
> > > along with avcodec.h is not ok)
> >
> > snow.h is included to get access to the DWTELEM #define. Would you have
> > any ideas on a better way of doing this?
> 
> Maybe right after DCTELEM in dsputil.h ?

yes, seems like the simplest solution ...

[...]

-- 
Michael





More information about the ffmpeg-devel mailing list