[FFmpeg-devel] [PATCH] Add test for vorbis encode

[Michael Niedermayer]

On Thu, May 05, 2016 at 05:03:00PM +0000, Petru Rares Sincraian wrote:
> Here is the patch,
> 
> Sorry.
> 
> On 05/05/16 19:01, Petru Rares Sincraian wrote:
> > Hi there,
> >
> > Here is a patch for the vorbis encoder. I only tried this test on a 
> > x64 architecture, maybe I need to change the FUZZ parameter.
> >
> > Can anybody test in another architecture?
> >
> >
> > Many thanks,
> > Petru Rares.
> 

>  vorbis.mak |    9 +++++++++
>  1 file changed, 9 insertions(+)
> bde8bbf92b64a31bac683a47431d89c2c2104021  0001-Add-test-for-vorbis-encoder.patch
> From b7d07c0a97919c11072d1210579432bebb2967c2 Mon Sep 17 00:00:00 2001
> From: Petru Rares Sincraian <psincraian at outlook.com>
> Date: Thu, 5 May 2016 18:56:38 +0200

> Subject: [PATCH] Add test for vorbis encoder

the first line of commit messages should start with the "area" or
file that is changed like "fate:"
fixed

> 
> ---
>  tests/fate/vorbis.mak | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/tests/fate/vorbis.mak b/tests/fate/vorbis.mak
> index 1344180..1d73fd3 100644
> --- a/tests/fate/vorbis.mak
> +++ b/tests/fate/vorbis.mak
> @@ -1,3 +1,12 @@
> +FATE_VORBIS += fate-vorbis-encode
> +fate-vorbis-encode: CMD = enc_dec_pcm ogg wav s16le $(TARGET_SAMPLES)/audio-reference/luckynight_2ch_44kHz_s16.wav -c:a vorbis -strict experimental

> +fate-vorbis-encode: CMP = stddev

this had to be moved to the end of the file as all the CMPs are
set there to oneoff so the CMP = staddev has no effect unless its
after that


> +fate-vorbis-encode: REF = $(SAMPLES)/audio-reference/luckynight_2ch_44kHz_s16.wav
> +fate-vorbis-encode: CMP_SHIFT = 0
> +fate-vorbis-encode: CMP_TARGET = 4810
> +fate-vorbis-encode: SIZE_TOLERANCE = 3560

> +fate-vorbis-encode: FUZZ = 0

tested, adjusted (due to MIPS), applied

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160505/80e9a144/attachment.sig>