On Sat, Jan 14, 2012 at 01:03:20PM -0500, Justin Ruggles wrote:
> 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
All applied.
> > 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.
Splitting will be done as the next step. I want to get this set in first.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
