On Sat, 27 Oct 2012, Justin Ruggles wrote:

The existing tests are renamed to fate-flac-16-* to allow different rules for
16-bit and 24-bit tests.
---
This will require upload of the 24-bit reference wav file to fate-suite.

tests/fate/flac.mak |   31 +++++++++++++++++++------------
1 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/tests/fate/flac.mak b/tests/fate/flac.mak
index dc49928..022be7a 100644
--- a/tests/fate/flac.mak
+++ b/tests/fate/flac.mak
@@ -1,17 +1,24 @@
-FATE_FLAC += fate-flac-chmode-indep                                     \
-             fate-flac-chmode-left_side                                 \
-             fate-flac-chmode-mid_side                                  \
-             fate-flac-chmode-right_side                                \
-             fate-flac-fixed                                            \
-             fate-flac-lpc-cholesky                                     \
-             fate-flac-lpc-levinson                                     \
+FATE_FLAC += fate-flac-16-chmode-indep                                  \
+             fate-flac-16-chmode-left_side                              \
+             fate-flac-16-chmode-mid_side                               \
+             fate-flac-16-chmode-right_side                             \
+             fate-flac-16-fixed                                         \
+             fate-flac-16-lpc-cholesky                                  \
+             fate-flac-16-lpc-levinson                                  \
+             fate-flac-24-comp-8                                        \

-fate-flac-chmode-%: OPTS = -ch_mode $(@:fate-flac-chmode-%=%)
-fate-flac-fixed: OPTS = -lpc_type fixed
-fate-flac-lpc-%: OPTS = -lpc_type $(@:fate-flac-lpc-%=%)
+fate-flac-16-chmode-%: OPTS = -ch_mode $(@:fate-flac-16-chmode-%=%)
+fate-flac-16-fixed: OPTS = -lpc_type fixed
+fate-flac-16-lpc-%: OPTS = -lpc_type $(@:fate-flac-16-lpc-%=%)
+
+fate-flac-16-%: REF = $(SAMPLES)/audio-reference/luckynight_2ch_44kHz_s16.wav
+fate-flac-16-%: CMD = enc_dec_pcm flac wav s16le $(REF) -c flac $(OPTS)
+
+fate-flac-24-comp-%: OPTS = -compression_level $(@:fate-flac-24-comp-%=%)
+
+fate-flac-24-%: REF = $(SAMPLES)/audio-reference/divertimenti_2ch_96kHz_s24.wav
+fate-flac-24-%: CMD = enc_dec_pcm flac wav s24le $(REF) -c flac $(OPTS)

-fate-flac-%: REF = $(SAMPLES)/audio-reference/luckynight_2ch_44kHz_s16.wav
-fate-flac-%: CMD = enc_dec_pcm flac wav s16le $(REF) -c flac $(OPTS)
fate-flac-%: CMP = oneoff
fate-flac-%: FUZZ = 0

--
1.7.1

This might be more readable if split in two steps, renaming the existing, and adding the new ones. Other than that, looks sensible to me.

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to