[Ffmpeg-devel] [patch] libavcodec/mpeg12.c:find_start_code() off by one

[Benjamin Larsson]

Diego Biurrun wrote:

>On Tue, Jan 10, 2006 at 10:48:19PM +0100, Michael Niedermayer wrote:
>  
>
>>patch looks ok, can be applied if the regression tests pass
>>    
>>
>
>Once again I vote for Benjamin.  Now he can even check out the
>regression tests ;)
>
>Diego
>
>  
>
Well current regressions test fail for me, that is with and without
this patch and with current cvs on a amd64 box.

--- ./data/ffmpeg.regression    2006-01-11 13:02:35.000000000 +0000
+++ /home/banan/cvs_ffmpeg/ffmpeg/tests/ffmpeg.regression.ref  
2006-01-08 18:11:57.000000000 +0000
@@ -119,10 +119,10 @@
 2653642 ./data/a-ffv1.avi
 799d3db687f6cdd7a837ec156efc171f *./data/out.yuv
 stddev:  0.00 PSNR:99.99 bytes:7602176
-c450be7c78afc98d15f4aed2f1c997a6 *./data/a-snow.avi
-1194368 ./data/a-snow.avi
-1f095d14dcb4fb105dba9bcceb9de777 *./data/out.yuv
-stddev:  2.90 PSNR:38.87 bytes:7602176
+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

So can someone confirm these regression checks?

MvH
Benjamin Larsson

-- 
"incorrect information" is an oxymoron. Information is, by definition, factual, correct.