On 12/29/2011 07:42 PM, Derek Buitenhuis wrote: >> --- >> tests/Makefile | 1 + >> tests/fate/s302m.mak | 6 ++++++ >> tests/ref/fate/s302m-nctapsa01hi2 | 1 + >> 3 files changed, 8 insertions(+), 0 deletions(-) >> create mode 100644 tests/fate/s302m.mak >> create mode 100644 tests/ref/fate/s302m-nctapsa01hi2 > > lol. I swear that file is the sole sample that exists for > s302m online. It makes yet another appearance! > >> diff --git a/tests/Makefile b/tests/Makefile >> index 229a576..aacd00f 100644 >> --- a/tests/Makefile >> +++ b/tests/Makefile >> @@ -47,6 +47,7 @@ include $(SRC_PATH)/tests/fate/mpc.mak >> include $(SRC_PATH)/tests/fate/prores.mak >> include $(SRC_PATH)/tests/fate/qtrle.mak >> include $(SRC_PATH)/tests/fate/real.mak >> +include $(SRC_PATH)/tests/fate/s302m.mak >> include $(SRC_PATH)/tests/fate/screen.mak >> include $(SRC_PATH)/tests/fate/utvideo.mak >> include $(SRC_PATH)/tests/fate/vorbis.mak > > Does s302m really warrant its own file? > >> diff --git a/tests/fate/s302m.mak b/tests/fate/s302m.mak >> new file mode 100644 >> index 0000000..efda6a9 >> --- /dev/null >> +++ b/tests/fate/s302m.mak >> @@ -0,0 +1,6 @@ >> +FATE_TESTS += fate-s302m-nctapsa01hi >> +fate-s302m-nctapsa01hi: CMD = pcm -i $(SAMPLES)/s302m/nctapsa01hi.mpg >> +fate-s302m-nctapsa01hi: REF = $(SAMPLES)/s302m/nctapsa01hi.s16 >> + >> +FATE_TESTS += fate-s302m-nctapsa01hi2 >> +fate-s302m-nctapsa01hi2: CMD = md5 -i $(SAMPLES)/s302m/nctapsa01hi.mpg -f >> s24le > > Sorry if I missed some discussion on IRC, but what's the point of a > separate test here?
the pcm test only compares 16-bit output currently. i figured an md5 test would be good to have here as well since the source is 20-bit. -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
