On 01/14/2012 12:32 PM, Diego Biurrun wrote:

> ---
[...]
>  rename tests/ref/fate/{truemotion1-15 => duck-tm1-15} (100%)
>  rename tests/ref/fate/{truemotion1-24 => duck-tm1-24} (100%)


the AVCodec name is "truemotion1" so i would prefer to have that in the
test name.

[...]
>  rename tests/ref/fate/{qt-msadpcm-stereo => msadpcm-stereo} (100%)
>  rename tests/ref/fate/{qt-msimaadpcm-stereo => msimaadpcm-stereo} (100%)


how about adpcm_ms-stereo and adpcm_ima_wav-stereo? those are the codec
names.

>  rename tests/ref/fate/{qt-rawpcm-16bit-stereo-signed-be => 
> pcm-16bit-stereo-signed-be} (100%)
>  rename tests/ref/fate/{qt-rawpcm-16bit-stereo-signed-le => 
> pcm-16bit-stereo-signed-le} (100%)
>  rename tests/ref/fate/{qt-rawpcm-8bit-mono-unsigned => 
> pcm-8bit-mono-unsigned} (100%)
>  rename tests/ref/fate/{qt-rawpcm-8bit-stereo-unsigned => 
> pcm-8bit-stereo-unsigned} (100%)


pcm_s16le-stereo, pcm_s16be-stereo, pcm_u8-mono, pcm_u8-stereo

>  rename tests/ref/fate/{pcm_dvd => pcm-dvd} (100%)


pcm_dvd is the name of the codec

>  rename tests/ref/fate/{ea-mad-pcm-planar => pcm-ea-mad} (100%)


this tests both pcm_s16le_planar audio and eamad video. we should split
the test instead.

>  rename tests/ref/fate/{film-cvid-pcm-stereo-8bit => pcm-film-stereo-8bit} 
> (100%)


film is the format. and this also tests cinepak, not just pcm_s8.

>  rename tests/ref/fate/{iff-pcm => pcm-iff} (100%)
>  rename tests/ref/fate/{w64 => pcm-w64} (100%)


these 2 are really more to test the demuxer, not pcm. they really should
be converted to demuxer tests. then the names make more sense as-is.

-Justin


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

Reply via email to