Diego Biurrun <[email protected]> writes:

> On Sun, Oct 07, 2012 at 07:39:09PM +0200, Janne Grunau wrote:
>> On 2012-10-07 19:22:30 +0200, Diego Biurrun wrote:
>> > 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 ...
>> 
>> AVCODEC_VC1DSP_H vs.
>> AVCODEC_X86_VC1DSP_H
>
> Not sure what you are trying to say there, let me be more explicit:
> Move the ff_vc1dsp_init_mmxext declaration to libavcodec/vc1dsp.h.

That would be counter the purpose of this patch.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to