[FFmpeg-devel] [PATCH] tests/fate-run: support both le/be formats on pixfmts

Hendrik Leppkes h.leppkes at gmail.com
Mon Jun 27 09:59:54 CEST 2016


On Mon, Jun 27, 2016 at 9:52 AM, Muhammad Faiz <mfcc64 at gmail.com> wrote:
> regardless of the actual supported formats of tested filters
> allowing filters to support only native endian formats
>
> Signed-off-by: Muhammad Faiz <mfcc64 at gmail.com>
> ---
>  tests/fate-run.sh                 |  7 ++++++-
>  tests/ref/fate/filter-pixfmts-lut | 22 ++++++++++++++++++++++
>  tests/ref/fate/filter-pixfmts-pad | 32 ++++++++++++++++++++++++++++++++
>  3 files changed, 60 insertions(+), 1 deletion(-)
>
> diff --git a/tests/fate-run.sh b/tests/fate-run.sh
> index c898695..066970c 100755
> --- a/tests/fate-run.sh
> +++ b/tests/fate-run.sh
> @@ -232,7 +232,12 @@ pixfmts(){
>      $showfiltfmts scale | awk -F '[ \r]' '/^OUTPUT/{ fmt=substr($3, 5); print fmt }' | sort >$scale_out_fmts
>      comm -12 $scale_in_fmts $scale_out_fmts >$scale_exclude_fmts
>
> -    $showfiltfmts $filter | awk -F '[ \r]' '/^INPUT/{ fmt=substr($3, 5); print fmt }' | sort >$in_fmts
> +    $showfiltfmts $filter | awk -F '[ \r]' \
> +        '/^INPUT/{ fmt=substr($3, 5);
> +        print fmt;
> +        print gensub(/(be)$/, "le", "g", fmt);
> +        print gensub(/(le)$/, "be", "g", fmt);
> +        }' | sort >$in_fmts
>      pix_fmts=$(comm -12 $scale_exclude_fmts $in_fmts)
>

This doesn't really test anything new, does it?
Just adds one more scaling step to convert endianness for some filters.

- Hendrik


More information about the ffmpeg-devel mailing list