On Mon, Jan 18, 2016 at 04:50:18PM +0100, Hendrik Leppkes wrote:
> On Mon, Jan 18, 2016 at 4:38 PM, Diego Biurrun <[email protected]> wrote:
> > ---
> >  libavcodec/x86/Makefile           |   2 +
> >  libavcodec/x86/hpeldsp.asm        |  89 ------------------------------
> >  libavcodec/x86/hpeldsp.h          |   4 ++
> >  libavcodec/x86/hpeldsp_init.c     |  25 ++-------
> >  libavcodec/x86/hpeldsp_vp3.asm    | 111 
> > ++++++++++++++++++++++++++++++++++++++
> >  libavcodec/x86/hpeldsp_vp3_init.c |  54 +++++++++++++++++++
> >  6 files changed, 174 insertions(+), 111 deletions(-)
> >  create mode 100644 libavcodec/x86/hpeldsp_vp3.asm
> >  create mode 100644 libavcodec/x86/hpeldsp_vp3_init.c
> >
> > [ ... snip ... ]

Please trim your quotes a bit, thank you.

> IMHO, either split this off entirely into its own DSP Context, or keep
> it grouped with the others.

Why?  VP3 depends on hpeldsp either way.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to