[libav-devel] [PATCH] fate: split acodec-pcm into individual tests

[Måns Rullgård]

Justin Ruggles <justin.ruggles at gmail.com> writes:

> this removes 2 redundant tests for pcm in mkv.
> we can add the coverage back in later as fate-lavf tests if needed.
> ---
>  tests/codec-regression.sh    |   50 ++++++++++++++++++++++++++----
>  tests/ref/acodec/pcm         |   68 ------------------------------------------
>  tests/ref/acodec/pcm_alaw    |    4 ++
>  tests/ref/acodec/pcm_f32be   |    4 ++
>  tests/ref/acodec/pcm_f32le   |    4 ++
>  tests/ref/acodec/pcm_f64be   |    4 ++
>  tests/ref/acodec/pcm_f64le   |    4 ++
>  tests/ref/acodec/pcm_mulaw   |    4 ++
>  tests/ref/acodec/pcm_s16be   |    4 ++
>  tests/ref/acodec/pcm_s16le   |    4 ++
>  tests/ref/acodec/pcm_s24be   |    4 ++
>  tests/ref/acodec/pcm_s24daud |    4 ++
>  tests/ref/acodec/pcm_s24le   |    4 ++
>  tests/ref/acodec/pcm_s32be   |    4 ++
>  tests/ref/acodec/pcm_s32le   |    4 ++
>  tests/ref/acodec/pcm_s8      |    4 ++
>  tests/ref/acodec/pcm_u8      |    4 ++
>  17 files changed, 103 insertions(+), 75 deletions(-)
>  delete mode 100644 tests/ref/acodec/pcm
>  create mode 100644 tests/ref/acodec/pcm_alaw
>  create mode 100644 tests/ref/acodec/pcm_f32be
>  create mode 100644 tests/ref/acodec/pcm_f32le
>  create mode 100644 tests/ref/acodec/pcm_f64be
>  create mode 100644 tests/ref/acodec/pcm_f64le
>  create mode 100644 tests/ref/acodec/pcm_mulaw
>  create mode 100644 tests/ref/acodec/pcm_s16be
>  create mode 100644 tests/ref/acodec/pcm_s16le
>  create mode 100644 tests/ref/acodec/pcm_s24be
>  create mode 100644 tests/ref/acodec/pcm_s24daud
>  create mode 100644 tests/ref/acodec/pcm_s24le
>  create mode 100644 tests/ref/acodec/pcm_s32be
>  create mode 100644 tests/ref/acodec/pcm_s32le
>  create mode 100644 tests/ref/acodec/pcm_s8
>  create mode 100644 tests/ref/acodec/pcm_u8

Looks OK, but...

> +# no compatible muxer or demuxer
> +# if [ -n "$do_pcm_u24be" ] ; then
> +# do_audio_enc_dec ??? u32 pcm_u24be
> +# fi
> +# if [ -n "$do_pcm_u24le" ] ; then
> +# do_audio_enc_dec ??? u32 pcm_u24le
> +# fi
[...]
> +# no compatible muxer or demuxer
> +# if [ -n "$do_pcm_u32be" ] ; then
> +# do_audio_enc_dec ??? u32 pcm_u32be
> +# fi
> +# if [ -n "$do_pcm_u32le" ] ; then
> +# do_audio_enc_dec ??? u32 pcm_u32le
> +# fi

Is this still true?

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