[FFmpeg-devel] [PATCH] Add a FATE test for the G.722 decoder

[Måns Rullgård]

Martin Storsj? <martin at martin.st> writes:

> Hi,
>
> As in $subj, an initial test for the decoder. If ok, I'll wait until 
> tomorrow before applying, so that the sample will have propagated out to 
> most fate machines.
>
> I'm still waiting for an OK from the ITU on redistributing the official 
> reference test vectors, if ok, I'll add them too.
>
> // Martin
>
> From 56a85741b66dceead6d51db97e68bf4000be825c Mon Sep 17 00:00:00 2001
> From: Martin Storsjo <martin at martin.st>
> Date: Mon, 26 Jul 2010 12:20:56 +0300
> Subject: [PATCH] Add a FATE test for the G.722 decoder
>
> ---
>  tests/fate2.mak          |    3 +
>  tests/ref/fate/g722dec-1 |  167 ++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 170 insertions(+), 0 deletions(-)
>  create mode 100644 tests/ref/fate/g722dec-1
>
> diff --git a/tests/fate2.mak b/tests/fate2.mak
> index f4c2cea..69ffeea 100644
> --- a/tests/fate2.mak
> +++ b/tests/fate2.mak
> @@ -127,6 +127,9 @@ fate-atrac3-3: REF = $(SAMPLES)/atrac3/mc_sich_at3_132_small.pcm
>  FATE_TESTS += fate-gsm
>  fate-gsm: CMD = framecrc -i $(SAMPLES)/gsm/ciao.wav
>  
> +FATE_TESTS += fate-g722dec-1
> +fate-g722dec-1: CMD = framecrc -ar 16000 -i $(SAMPLES)/g722/conf-adminmenu-162.g722

Why is the -ar option needed?  Is the sample rate not signalled in the
file at all?

-- 
M?ns Rullg?rd
mans at mansr.com