On Tue, May 27, 2014 at 08:22:29AM -0700, Diego Biurrun wrote: > --- > libavcodec/dsputil.c | 12 ------ > libavcodec/dsputil.h | 3 -- > libavcodec/ppc/Makefile | 1 + > libavcodec/ppc/int_altivec.c | 48 +----------------------- > libavcodec/ppc/svq1enc_altivec.c | 80 > ++++++++++++++++++++++++++++++++++++++++ > libavcodec/svq1enc.c | 58 ++++++++++------------------- > libavcodec/svq1enc.h | 78 +++++++++++++++++++++++++++++++++++++++ > libavcodec/x86/Makefile | 1 + > libavcodec/x86/dsputilenc_mmx.c | 36 ------------------ > libavcodec/x86/svq1enc_mmx.c | 72 ++++++++++++++++++++++++++++++++++++ > libavutil/ppc/util_altivec.h | 3 ++ > 11 files changed, 255 insertions(+), 137 deletions(-) > create mode 100644 libavcodec/ppc/svq1enc_altivec.c > create mode 100644 libavcodec/svq1enc.h > create mode 100644 libavcodec/x86/svq1enc_mmx.c > [...] > diff --git a/libavcodec/ppc/int_altivec.c b/libavcodec/ppc/int_altivec.c > index 42c396c..fa3cb66 100644 > --- a/libavcodec/ppc/int_altivec.c > +++ b/libavcodec/ppc/int_altivec.c > @@ -30,54 +30,10 @@ > > #include "libavutil/attributes.h" > #include "libavutil/ppc/types_altivec.h" > +#include "libavutil/ppc/util_altivec.h"
somehow this looks like a stray include > #include "libavcodec/dsputil.h" > #include "dsputil_altivec.h" > [...] Moving SVQ1EncContext to a header file was a bit surprising too but in general patch might be OK. _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel