On Tue, 16 Apr 2013 18:09:19 +0200, Diego Biurrun <[email protected]> wrote:
> ---
>  tests/fate-run.sh                                |    8 ++++++++
>  tests/fate/avfilter.mak                          |    3 +--
>  tests/fate/filter-video.mak                      |    4 ++++
>  tests/lavfi-regression.sh                        |    7 -------
>  tests/ref/{lavfi/pixdesc => fate/filter-pixdesc} |    0
>  5 files changed, 13 insertions(+), 9 deletions(-)
>  rename tests/ref/{lavfi/pixdesc => fate/filter-pixdesc} (100%)
> 
> diff --git a/tests/fate-run.sh b/tests/fate-run.sh
> index 72554ad..6e30b5a 100755
> --- a/tests/fate-run.sh
> +++ b/tests/fate-run.sh
> @@ -171,6 +171,14 @@ video_filter(){
>          $FLAGS $ENC_OPTS -vf "$filters" -vcodec rawvideo $* -f nut md5:
>  }
>  
> +pixdesc(){
> +    pix_fmts="$(avconv -pix_fmts list 2>/dev/null | awk 'NR > 8 && /^IO/ { 
> print $2 }' | sort)"
> +    for pix_fmt in $pix_fmts; do
> +        test=$pix_fmt
> +        video_filter "format=$pix_fmt,pixdesctest" -pix_fmt $pix_fmt
> +    done
> +}
> +
>  mkdir -p "$outdir"
>  
>  exec 3>&2
> diff --git a/tests/fate/avfilter.mak b/tests/fate/avfilter.mak
> index e31f449..edca861 100644
> --- a/tests/fate/avfilter.mak
> +++ b/tests/fate/avfilter.mak
> @@ -1,5 +1,4 @@
> -FATE_LAVFI = fate-lavfi-pixdesc                                         \
> -             fate-lavfi-pixfmts_copy                                    \
> +FATE_LAVFI = fate-lavfi-pixfmts_copy                                    \
>               fate-lavfi-pixfmts_crop                                    \
>               fate-lavfi-pixfmts_hflip                                   \
>               fate-lavfi-pixfmts_null                                    \
> diff --git a/tests/fate/filter-video.mak b/tests/fate/filter-video.mak
> index d575375..36e9f02 100644
> --- a/tests/fate/filter-video.mak
> +++ b/tests/fate/filter-video.mak
> @@ -77,6 +77,10 @@ FATE_FILTER_VSYNTH-$(CONFIG_VFLIP_FILTER) += 
> fate-filter-vflip_vflip
>  fate-filter-vflip_vflip: CMD = video_filter "vflip,vflip"
>  
>  
> +FATE_FILTER_VSYNTH-$(CONFIG_FORMAT_FILTER) += fate-filter-pixdesc
> +fate-filter-pixdesc: CMD = pixdesc
> +
> +
>  $(FATE_FILTER_VSYNTH-yes): $(VREF)
>  $(FATE_FILTER_VSYNTH-yes): SRC = $(TARGET_PATH)/tests/vsynth1/%02d.pgm
>  
> diff --git a/tests/lavfi-regression.sh b/tests/lavfi-regression.sh
> index f2fc752..588a53f 100755
> --- a/tests/lavfi-regression.sh
> +++ b/tests/lavfi-regression.sh
> @@ -50,13 +50,6 @@ do_lavfi_pixfmts "pad"     "500:400:20:20"
>  do_lavfi_pixfmts "scale"   "200:100"
>  do_lavfi_pixfmts "vflip"   ""
>  
> -if [ -n "$do_pixdesc" ]; then
> -    pix_fmts="$($avconv -pix_fmts list 2>/dev/null | awk 'NR > 8 && /^IO/ { 
> print $2 }' | sort)"
> -    for pix_fmt in $pix_fmts; do
> -        do_video_filter $pix_fmt "format=$pix_fmt,pixdesctest" -pix_fmt 
> $pix_fmt
> -    done
> -fi
> -
>  # TODO: add tests for
>  # direct rendering,
>  # chains with feedback loops
> diff --git a/tests/ref/lavfi/pixdesc b/tests/ref/fate/filter-pixdesc
> similarity index 100%
> rename from tests/ref/lavfi/pixdesc
> rename to tests/ref/fate/filter-pixdesc
> -- 
> 1.7.9.5
> 

Would be even nicer to make each of those into a separate test, but that's for
later.

Patch looks fine.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to