[FFmpeg-cvslog] r24134 - in trunk: Makefile tests/fate-run.sh

[Michael Niedermayer]

On Fri, Jul 09, 2010 at 02:33:18AM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Fri, Jul 09, 2010 at 02:40:51AM +0200, mru wrote:
> >> Author: mru
> >> Date: Fri Jul  9 02:40:51 2010
> >> New Revision: 24134
> >> 
> >> Log:
> >> fate: add oneoff comparison method
> >> 
> >> This adds the 'oneoff' comparison method.  It compares two s16le pcm
> >> files with a max difference <= the fuzz factor treated as success.
> >> Default fuzz is 1.  On failure, the full tiny_psnr output is printed.
> >
> > it also might make sense to bound the sum of squared differences, and not
> > just the maximum. A file having 1% of samples differing +-1 and one that is
> > in 50% of samples +-1 off are not the same thing
> 
> With different floating-point rounding modes, it is perfectly normal
> to have all negative samples off by one (round towards zero vs -inf).
> That is the type of test this is intended for.

actually, floating point decoders should output floating point samples
once all the issues with that have been solved.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100709/7383669e/attachment.pgp>