On Sun, Oct 07, 2012 at 06:50:44PM +0200, Janne Grunau wrote: > > --- /dev/null > +++ b/libavcodec/x86/vc1dsp.h > @@ -0,0 +1,35 @@ > + > +#ifndef AVCODEC_X86_VC1DSP_H > +#define AVCODEC_X86_VC1DSP_H > + > +#include "libavcodec/vc1dsp.h" > + > +void ff_vc1dsp_init_mmx(VC1DSPContext *dsp); > +void ff_vc1dsp_init_mmxext(VC1DSPContext *dsp); > + > +#endif /* AVCODEC_X86_VC1DSP_H */
Huh? This is already declared in libavcodec/vc1dsp.h ... > --- /dev/null > +++ b/libavcodec/x86/vc1dsp_init.c > @@ -0,0 +1,124 @@ > + > +#include "libavutil/cpu.h" > +#include "libavutil/x86/cpu.h" > +#include "libavcodec/vc1dsp.h" > +#include "vc1dsp.h" > + > +#include "config.h" nit: drop the empty line Looks promising otherwise. While you're dealing with VC-1 x86 init, you could have a look at the parts in dsputil_mmx.c ;) Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
