[libav-devel] [PATCH] fate: employ better names and add a convenient shorthand for vp6 tests

Måns Rullgård mans at mansr.com
Wed May 9 15:34:48 CEST 2012


Diego Biurrun <diego at biurrun.de> writes:

> ---
>  tests/fate/vpx.mak               |   22 +++++++++++-----------
>  tests/ref/fate/{ea-vp60 => vp60} |    0
>  tests/ref/fate/{ea-vp61 => vp61} |    0
>  3 files changed, 11 insertions(+), 11 deletions(-)
>  rename tests/ref/fate/{ea-vp60 => vp60} (100%)
>  rename tests/ref/fate/{ea-vp61 => vp61} (100%)
>
> diff --git a/tests/fate/vpx.mak b/tests/fate/vpx.mak
> index e18caf9..9b0be7a 100644
> --- a/tests/fate/vpx.mak
> +++ b/tests/fate/vpx.mak
> @@ -1,12 +1,3 @@
> -FATE_EA_VP6 += fate-ea-vp60
> -fate-ea-vp60: CMD = framecrc -i $(SAMPLES)/ea-vp6/g36.vp6
> -
> -FATE_EA_VP6 += fate-ea-vp61
> -fate-ea-vp61: CMD = framecrc -i $(SAMPLES)/ea-vp6/MovieSkirmishGondor.vp6 -t 4
> -
> -FATE_AVCONV += $(FATE_EA_VP6)
> -fate-ea-vp6: $(FATE_EA_VP6)
> -
>  FATE_VP3 += fate-vp31
>  fate-vp31: CMD = framecrc -i $(SAMPLES)/vp3/vp31.avi
>
> @@ -19,12 +10,21 @@ fate-vp3: $(FATE_VP3)
>  FATE_AVCONV += fate-vp5
>  fate-vp5: CMD = framecrc -i $(SAMPLES)/vp5/potter512-400-partial.avi -an
>
> -FATE_AVCONV += fate-vp6a
> +FATE_VP6 += fate-vp60
> +fate-vp60: CMD = framecrc -i $(SAMPLES)/ea-vp6/g36.vp6
> +
> +FATE_VP6 += fate-vp61
> +fate-vp61: CMD = framecrc -i $(SAMPLES)/ea-vp6/MovieSkirmishGondor.vp6 -t 4
> +
> +FATE_VP6 += fate-vp6a
>  fate-vp6a: CMD = framecrc -i $(SAMPLES)/flash-vp6/300x180-Scr-f8-056alpha.flv
>
> -FATE_AVCONV += fate-vp6f
> +FATE_VP6 += fate-vp6f
>  fate-vp6f: CMD = framecrc -i $(SAMPLES)/flash-vp6/clip1024.flv
>
> +FATE_AVCONV += $(FATE_VP6)
> +fate-vp6: $(FATE_VP6)
> +
>  VP8_SUITE = 001 002 003 004 005 006 007 008 009 010 011 012 013 014 015 016 017
>
>  define FATE_VP8_SUITE
> diff --git a/tests/ref/fate/ea-vp60 b/tests/ref/fate/vp60
> similarity index 100%
> rename from tests/ref/fate/ea-vp60
> rename to tests/ref/fate/vp60
> diff --git a/tests/ref/fate/ea-vp61 b/tests/ref/fate/vp61
> similarity index 100%
> rename from tests/ref/fate/ea-vp61
> rename to tests/ref/fate/vp61
> -- 

What are vp60 and vp61 supposed to signify?  Are these different
variants of the codec or just two numbered tests of the same thing?  In
the latter case, a hyphen should be inserted between the codec and the
test number like for other such 

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


More information about the libav-devel mailing list