reftests: Pass common_cflags

diff --git a/testsuite/gsk/reftest-compare.c b/testsuite/gsk/reftest-compare.c
index 21f5962d9c47d4cede9f1950ff88200586103611..6cb8af7fef51395bed8987bf12b4e94e4e329fbb 100644 (file)
--- a/testsuite/gsk/reftest-compare.c
+++ b/testsuite/gsk/reftest-compare.c
@@ -120,15 +120,15 @@ buffer_diff_core (const guchar *buf_a,
             {
               int value_a = (row_a[x] >> (channel*8)) & 0xff;
               int value_b = (row_b[x] >> (channel*8)) & 0xff;
-              guint diff;
-
-              diff = ABS (value_a - value_b);
-              diff *= 4;  /* emphasize */
-              if (diff)
-                diff += 128; /* make sure it's visible */
-              if (diff > 255)
-                diff = 255;
-              diff_pixel |= diff << (channel*8);
+              guint channel_diff;
+
+              channel_diff = ABS (value_a - value_b);
+              channel_diff *= 4;  /* emphasize */
+              if (channel_diff)
+                channel_diff += 128; /* make sure it's visible */
+              if (channel_diff > 255)
+                channel_diff = 255;
+              diff_pixel |= channel_diff << (channel*8);
             }
 
           if ((diff_pixel & 0x00ffffff) == 0)

diff --git a/testsuite/reftests/meson.build b/testsuite/reftests/meson.build
index e85699f0f73eb78b629ac52c8bcce7473ee786db..16bfa5231b806a9d2e6ff984161e13886f7edacf 100644 (file)
--- a/testsuite/reftests/meson.build
+++ b/testsuite/reftests/meson.build
@@ -1,7 +1,17 @@
 # FIXME: $(no_undefined) ?
+
+reftest_cflags = []
+foreach flag: common_cflags
+  if flag not in ['-Werror=missing-prototypes', '-Werror=missing-declarations', '-fvisibility=hidden']
+    reftest_cflags += flag
+  endif
+endforeach
+
+
 libgtkreftestprivate = library('gtkreftestprivate',
   'reftest-compare.c', 'reftest-module.c', 'reftest-snapshot.c',
-  dependencies : libgtk_dep)
+  dependencies : libgtk_dep,
+  c_args: reftest_cflags)
 
 libreftest = library('reftest',
   [ 'expand-expander.c',
@@ -14,6 +24,7 @@ libreftest = library('reftest',
     'animation-direction.c',
   ],
   link_with: libgtkreftestprivate,
+  c_args: reftest_cflags,
   dependencies : libgtk_dep)
 
 tcdata = configuration_data()

diff --git a/testsuite/reftests/reftest-compare.c b/testsuite/reftests/reftest-compare.c
index dc422089cd230dc80511ac9cca501b792b7d8afc..651eac9ca967136e616a3570dc3bc715645af60a 100644 (file)
--- a/testsuite/reftests/reftest-compare.c
+++ b/testsuite/reftests/reftest-compare.c
@@ -121,15 +121,15 @@ buffer_diff_core (const guchar *buf_a,
             {
               int value_a = (row_a[x] >> (channel*8)) & 0xff;
               int value_b = (row_b[x] >> (channel*8)) & 0xff;
-              guint diff;
-
-              diff = ABS (value_a - value_b);
-              diff *= 4;  /* emphasize */
-              if (diff)
-                diff += 128; /* make sure it's visible */
-              if (diff > 255)
-                diff = 255;
-              diff_pixel |= diff << (channel*8);
+              guint channel_difff;
+
+              channel_difff = ABS (value_a - value_b);
+              channel_difff *= 4;  /* emphasize */
+              if (channel_difff)
+                channel_difff += 128; /* make sure it's visible */
+              if (channel_difff > 255)
+                channel_difff = 255;
+              diff_pixel |= channel_difff << (channel * 8);
             }
 
           if ((diff_pixel & 0x00ffffff) == 0)

diff --git a/testsuite/reftests/reftest-module.c b/testsuite/reftests/reftest-module.c
index acab3595227efc59e017d0a6216717189b6913b5..d47243edd2bc7818271a5296f27a43604fcfc233 100644 (file)
--- a/testsuite/reftests/reftest-module.c
+++ b/testsuite/reftests/reftest-module.c
@@ -54,7 +54,7 @@ reftest_module_new_take (GModule *module,
   if (all_modules == NULL)
     all_modules = g_hash_table_new (g_str_hash, g_str_equal);
 
-  g_hash_table_insert (all_modules, filename ? filename : "", result);
+  g_hash_table_insert (all_modules, filename ? filename : g_strdup (""), result);
 
   return result;
 }