[Ffmpeg-devel] [patch] libavcodec/mpeg12.c:find_start_code() off by one
Aurelien Jacobs
aurel
Thu Jan 12 22:24:09 CET 2006
On Thu, 12 Jan 2006 21:12:38 +0100
Michael Niedermayer <michaelni at gmx.at> wrote:
> Hi
>
> On Thu, Jan 12, 2006 at 08:50:11PM +0100, Aurelien Jacobs wrote:
> > On Thu, 12 Jan 2006 17:11:16 +0100
> > Michael Niedermayer <michaelni at gmx.at> wrote:
> >
> > > Hi
> > >
> > > On Thu, Jan 12, 2006 at 03:31:55PM +0100, Diego Biurrun wrote:
> > > > On Wed, Jan 11, 2006 at 02:39:27PM +0100, Diego Biurrun wrote:
> > > > > On Wed, Jan 11, 2006 at 01:18:48PM +0000, Benjamin Larsson wrote:
> > > > > >
> > > > > > Well current regressions test fail for me, that is with and without
> > > > > > this patch and with current cvs on a amd64 box.
> > > > >
> > > > > Hmm, do the regression tests work under AMD64 at all? The last time I
> > > > > tested they failed on PPC. I'll report later.
> > > >
> > > > Yes, they fail on PPC on a vanilla source tree, this is the end of the
> > > > output of 'make test':
> > > >
> > > > ../ffmpeg_g -y -flags +bitexact -dct fastint -idct simple -y -i
> > > > ./data/a-adpcm_yam.wav -f wav ./data/out.wav
> > > > FFmpeg version CVS, Copyright (c) 2000-2004 Fabrice Bellard
> > > > configuration: --enable-gpl
> > > > libavcodec version: 51.0.0
> > > > libavformat version: 50.0.0
> > > > built on Jan 12 2006 15:14:39, gcc: 4.0.3 20060104 (prerelease)
> > > > (Debian 4.0.2-6)
> > > > size= 1032kB time=6.0 bitrate=1411.3kbits/s
> > > > --- ./data/ffmpeg.regression 2006-01-12 15:22:01.000000000 +0100
> > > > +++ /home/diego/src/ffmpeg/tests/ffmpeg.regression.ref 2006-01-08
> > > > 22:32:12.000000000 +0100
> > > > @@ -119,12 +119,12 @@
> > > > 2653642 ./data/a-ffv1.avi
> > > > 799d3db687f6cdd7a837ec156efc171f *./data/out.yuv
> > > > stddev: 0.00 PSNR:99.99 bytes:7602176
> > > > -472a891a0d07e050943d62504240c50f *./data/a-snow.avi
> > > > -1362158 ./data/a-snow.avi
> > > > -946ff31ac244e913452ccac77e27be1b *./data/out.yuv
> > > > -stddev: 3.00 PSNR:38.56 bytes:7602176
> > > > -745a5ad480b5e2a2319be85764b89012 *./data/a-snow53.avi
> > > > -3540248 ./data/a-snow53.avi
> > > > +14fa515bf25a47fc2ad3d18b726e8e31 *./data/a-snow.avi
> > > > +1197138 ./data/a-snow.avi
> > > > +e7c746171b092266b0cf55bb5de2a40a *./data/out.yuv
> > > > +stddev: 2.89 PSNR:38.87 bytes:7602176
> > > > +11fd61ee7e67ef7a7b2a3df973691305 *./data/a-snow53.avi
> > > > +3533710 ./data/a-snow53.avi
> > > > 799d3db687f6cdd7a837ec156efc171f *./data/out.yuv
> > > > stddev: 0.00 PSNR:99.99 bytes:7602176
> > > > e1da20e3f52f4d2aa18e9486986161fc *./data/a-dv.dv
> > > >
> > > > Regression test: Error.
> > > > make[1]: *** [codectest] Error 1
> > > > make[1]: Leaving directory `/home/diego/src/ffmpeg/tests'
> > > > make: *** [test] Error 2
> > > >
> > > > The other tests fail at some point at well. So are the regression test
> > > > i386-only?
> > >
> > > no they where working fine on non x86 sometime ago but people apparently just
> > > ignore it if they fail instead of reporting ...
> > > maybe someone could do a binary search to find out when they broke (just
> > > test a-snow53.avi might be a good idea too to speed this up)
> >
> > Well. I just did it on amd64. The snow regression test failure was
> > introduced by the following commit :
> >
> > Sat, 7 Jan 2006 16:53:24 +0100 (CET)
> > regression test for iterative ME & wavelet cmp functions
> >
> > Wich essentially contain the following:
> >
> > -do_ffmpeg $file -y -strict -2 -f pgmyuv -i $raw_src -an -vcodec snow -qscale 2 $file
> > +do_ffmpeg $file -y -strict -2 -f pgmyuv -i $raw_src -an -vcodec snow -qscale 2 -flags +qpel -me iter -dia_size 2 -cmp 12 -subcmp 12 $file
>
> ok thanks ...
I've done some more tests. The '-cmp 12' option is responsible for the different
md5 on x86 and x86-64.
Aurel
More information about the ffmpeg-devel
mailing list