[FFmpeg-devel] [PATCH]Enable ac3 decoding regression test (improved PSNR)

[Carl Eugen Hoyos]

Hi!

Carl Eugen Hoyos <cehoyos <at> ag.or.at> writes:

> > > Attached is a patch to enable ac3 decoding regression tests. I found the 
> > > best values for stddec and PSNR with the following command line:
> > > tiny_psnr data/ac3.vsynth.out.wav data/vsynth.ref.wav 2 1024
> > >
> > > Please comment, Carl Eugen
> > 
> > patch looks ok
> 
> Patch applied, Carl Eugen

As can be seen on FATE, http://fate.multimedia.cx/index.php?build_record=33798
this patch broke regression tests with gcc 2.95.3.
I think the patch should not be reverted, because it shows that ffmpeg decodes
AC-3 (slightly) different when compiled with gcc 2.95.3.

Carl Eugen

diff -u -w tests/ffmpeg.regression.ref   tests/data/vsynth.regression
--- /tests/ffmpeg.regression.ref   2008-10-13 11:11:41.000000000 +0200
+++ tests/data/vsynth.regression        2008-10-14 17:00:00.000000000 +0200
@@ -177,9 +177,9 @@
 stddev: 4394.71 PSNR: 23.46 bytes:  1053352/  1058444
 aefe11ab5067621a1c674859d6413891 *./tests/data/a-ac3.rm
 98203 ./tests/data/a-ac3.rm
-39d5646cba5a6cfedfd58845e6df3c06 *./tests/data/ac3.vsynth.out.wav
-stddev:10496.20 PSNR: 15.90 bytes:  1056812/  1058444
-stddev: 4631.51 PSNR: 23.00 bytes:  1055788/  1058444
+bf1867eb6c92e413b4e2a860c5a0e8ee *./tests/data/ac3.vsynth.out.wav
+stddev:10496.34 PSNR: 15.90 bytes:  1056812/  1058444
+stddev: 4631.55 PSNR: 23.00 bytes:  1055788/  1058444
 39878597b1d65cce473639a7d8c93b02 *./tests/data/a-g726.wav
 24279 ./tests/data/a-g726.wav
 888f2f016e608d044a1bacbca5497ed2 *./tests/data/g726.vsynth.out.wav