[Ffmpeg-cvslog] r6213 - in trunk: Changelog MAINTAINERS doc/ffmpeg-doc.texi libavcodec/Makefile libavcodec/allcodecs.c libavcodec/avcodec.h libavcodec/vp5.c libavcodec/vp56.c libavcodec/vp56.h libavcodec/vp56data.c libavcodec/vp56data.h libavcodec/vp5data.h libavcodec/vp6.c libavcodec/vp6data.h libavformat/flvdec.c libavformat/nsvdec.c libavformat/riff.c libavformat/swf.c

[Diego Biurrun]

On Mon, Sep 11, 2006 at 12:45:35AM +0200, Aurelien Jacobs wrote:
> On Sun, 10 Sep 2006 18:42:13 +0200
> Luca Barbato <lu_zero at gentoo.org> wrote:
> 
> > Aurelien Jacobs wrote:
> > > 
> > > What alignment is needed exactly ?
> > 
> > 16byte
> > 
> > > The only h264 function used is biweight_h264_pixels_tab[3].
> > 
> > 
> > Lets try to summarize
> > 
> > you use
> > 
> > put_pixels_tab[0][0]  ((line_size % 16) == 0)
> > put_pixels_tab[1][0]  unimplemented
> > put_no_rnd_pixels_l2[1] unimplemented
> > idct_put aligned input required
> > idct_add idem
> > biweight_h264_pixels_tab[3] unimplemented
> > 
> > which is the faulty one? ^^
> 
> As there is no altivec implementation of vp3 idct, it must be
> put_pixels_tab[0][0].
> Could you confirm it, testing with the following patch ?

Bingo, problem solved.

Diego