On Sun, May 13, 2012 at 04:46:50PM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > > On Fri, May 11, 2012 at 06:35:32PM +0100, Mans Rullgard wrote: > >> These two files use the same audio codec so only one test for > >> this is needed. > >> > >> Signed-off-by: Mans Rullgard <[email protected]> > >> --- > >> tests/fate/demux.mak | 6 -- > >> tests/fate/dpcm.mak | 3 + > >> tests/fate/video.mak | 6 ++ > >> tests/ref/fate/dpcm-interplay | 125 > >> ++++++++++++++++++++++++++++++++++++ > >> tests/ref/fate/interplay-mve-16bit | 80 ----------------------- > >> tests/ref/fate/interplay-mve-8bit | 125 > >> ------------------------------------ > >> 6 files changed, 134 insertions(+), 211 deletions(-) > >> create mode 100644 tests/ref/fate/dpcm-interplay > > > > The audio tracks have slightly different parameters: > > > > interplay-logo-2MB.mve > > Stream #0.1: Audio: interplay_dpcm, 22050 Hz, 2 channels, s16, 352 kb/s > > descent3-level5-16bit-partial.mve > > Stream #0.1: Audio: interplay_dpcm, 44100 Hz, 2 channels, s16, 705 kb/s > > > > I worry we might be reducing test coverage here... > > The sample rate is not used by the decoder.
Patch OK with me then. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
