commit f7b67565c195de14edcf28010582282fcd4a61d0
author Sebastien Alaiwan <sebastien.alaiwan@allegrodvt.com> Wed Jun 21 16:50:55 2017 +0200
committer Sebastien Alaiwan <sebastien.alaiwan@allegrodvt.com> Mon Jul 03 08:05:20 2017 +0000
tree 17153f0dcdb6127a5eb9e5b13fad6084428571ed
parent 910f479be48188e4331f22252a385f3c1593f105

Fix warning in test code

And ensure no other warnings will creep in by using
an 'if' statement instead of a #if.

Change-Id: Ic0ccc481d375de3b3cedc7e3e5530f6f9b011c37

av1/av1_cx_iface.c

diff --git a/av1/av1_cx_iface.c b/av1/av1_cx_iface.c
index 99d070b..2a8cee7 100644
--- a/av1/av1_cx_iface.c
+++ b/av1/av1_cx_iface.c
@@ -1013,7 +1013,8 @@
 }
 
 // Turn on to test if supplemental superframe data breaks decoding
-// #define TEST_SUPPLEMENTAL_SUPERFRAME_DATA
+#define TEST_SUPPLEMENTAL_SUPERFRAME_DATA 0
+
 static int write_superframe_index(aom_codec_alg_priv_t *ctx) {
   uint8_t marker = 0xc0;
   size_t max_frame_sz = 0;
@@ -1042,19 +1043,18 @@
   uint8_t buffer[256];
   uint8_t *x = buffer;
 
-#ifdef TEST_SUPPLEMENTAL_SUPERFRAME_DATA
-  uint8_t marker_test = 0xc0;
-  int mag_test = 2;     // 1 - 4
-  int frames_test = 4;  // 1 - 8
-  int index_sz_test = 2 + mag_test * frames_test;
-  marker_test |= frames_test - 1;
-  marker_test |= (mag_test - 1) << 3;
-  *x++ = marker_test;
-  for (int i = 0; i < mag_test * frames_test; ++i)
-    *x++ = 0;  // fill up with arbitrary data
-  *x++ = marker_test;
-  printf("Added supplemental superframe data\n");
-#endif
+  if (TEST_SUPPLEMENTAL_SUPERFRAME_DATA) {
+    uint8_t marker_test = 0xc0;
+    int mag_test = 2;     // 1 - 4
+    int frames_test = 4;  // 1 - 8
+    marker_test |= frames_test - 1;
+    marker_test |= (mag_test - 1) << 3;
+    *x++ = marker_test;
+    for (int i = 0; i < mag_test * frames_test; ++i)
+      *x++ = 0;  // fill up with arbitrary data
+    *x++ = marker_test;
+    printf("Added supplemental superframe data\n");
+  }
 
   *x++ = marker;
   for (int i = 0; i < ctx->pending_frame_count - 1; i++) {