On 11/30/2011 06:50 PM, Måns Rullgård wrote: > Justin Ruggles <[email protected]> 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?
yes it is as far as i know. but i'm working on adding support in mov, which is why i left them there. i did forget to update seek references though. fate didn't fail because i hadn't done make clean. i'll send an upated patch with that. -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
