On 12/30/2011 05:28 PM, Måns Rullgård wrote: > > > Diego Biurrun <[email protected]> wrote: > >> On Sat, Dec 31, 2011 at 02:06:03AM +0530, Gaurav Narula wrote: >>> >>> --- a/tests/fate-run.sh >>> +++ b/tests/fate-run.sh >>> @@ -69,6 +69,14 @@ md5(){ >>> >>> +pcm24(){ >>> + avconv "$@" -vn -f s24le - >>> +} >>> + >>> +pcm32(){ >>> + avconv "$@" -vn -f s32le - >>> +} >>> + >>> pcm(){ >>> avconv "$@" -vn -f s16le - >>> } >> >> This is not a test for a 302M decoder, it's infrastructure for testing >> 24 and 32 bit PCM in FATE. It's very welcome, but you need to split >> this patch into two pieces. > > I'm afraid there's more to it than that. Please don't commit this until I've > had a closer look.
This should work fine if the test uses CMP = diff. The oneoff tests obviously won't work until we add 24-bit and 32-bit support in tiny_psnr. -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
