testsuite: Fix various compiler warnings
authorMatthias Clasen <mclasen@redhat.com>
Fri, 13 Jan 2023 18:15:05 +0000 (13:15 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Sat, 14 Jan 2023 05:14:43 +0000 (00:14 -0500)
These are mostly dead stores that clang
isn't happy with.

testsuite/gtk/bitset.c
testsuite/gtk/recentmanager.c
testsuite/gtk/shortcuts.c

index 655d95b110a6c3ec8a3373fd85237a3a0b58f668..dc06d80233c0e4d9d6cbc2a9a5b3a4a4dac2b24d 100644 (file)
@@ -500,6 +500,7 @@ test_iter (void)
   g_assert_false (gtk_bitset_iter_next (&iter, &value));
 
   ret = gtk_bitset_iter_init_at (&iter, set, 0, &value);
+  g_assert_false (ret);
 
   g_assert_false (gtk_bitset_iter_is_valid (&iter));
   g_assert_cmpuint (gtk_bitset_iter_get_value (&iter), ==, 0);
index dbbce1142ec3b47a56837f01d7adbd6984026df0..df38d9e9ef5bb32c5f844386b5cbe8f6b3561214 100644 (file)
@@ -55,9 +55,8 @@ recent_manager_add (void)
   recent_data->app_exec = (char *)"testrecentchooser %u";
   if (g_test_trap_fork (0, G_TEST_TRAP_SILENCE_STDERR))
     {
-      res = gtk_recent_manager_add_full (manager,
-                                         uri,
-                                         recent_data);
+      res = gtk_recent_manager_add_full (manager, uri, recent_data);
+      g_assert_false (res);
     }
   g_test_trap_assert_failed ();
 
@@ -67,9 +66,8 @@ recent_manager_add (void)
   recent_data->app_exec = (char *)"testrecentchooser %u";
   if (g_test_trap_fork (0, G_TEST_TRAP_SILENCE_STDERR))
     {
-      res = gtk_recent_manager_add_full (manager,
-                                         uri,
-                                         recent_data);
+      res = gtk_recent_manager_add_full (manager, uri, recent_data);
+      g_assert_false (res);
     }
   g_test_trap_assert_failed ();
 
@@ -79,9 +77,8 @@ recent_manager_add (void)
   recent_data->app_exec = NULL;
   if (g_test_trap_fork (0, G_TEST_TRAP_SILENCE_STDERR))
     {
-      res = gtk_recent_manager_add_full (manager,
-                                         uri,
-                                         recent_data);
+      res = gtk_recent_manager_add_full (manager, uri, recent_data);
+      g_assert_false (res);
     }
   g_test_trap_assert_failed ();
 
@@ -90,9 +87,7 @@ recent_manager_add (void)
   recent_data->mime_type = (char *)"text/plain";
   recent_data->app_name = (char *)"testrecentchooser";
   recent_data->app_exec = (char *)"testrecentchooser %u";
-  res = gtk_recent_manager_add_full (manager,
-                                     uri,
-                                     recent_data);
+  res = gtk_recent_manager_add_full (manager, uri, recent_data);
   g_assert_true (res);
 
   g_slice_free (GtkRecentData, recent_data);
index ad5e9c798b0d24657b7b44842f4509211879683c..10a7913faaa4a4e9a0b01751cce7ca0fede1367e 100644 (file)
@@ -39,6 +39,8 @@ test_trigger_basic (void)
 
   trigger = gtk_never_trigger_get ();
 
+  g_assert_true (GTK_IS_SHORTCUT_TRIGGER (trigger));
+
   trigger = gtk_keyval_trigger_new (GDK_KEY_a, GDK_CONTROL_MASK);
   g_assert_cmpint (gtk_keyval_trigger_get_keyval (GTK_KEYVAL_TRIGGER (trigger)), ==, GDK_KEY_a);
   g_assert_cmpint (gtk_keyval_trigger_get_modifiers (GTK_KEYVAL_TRIGGER (trigger)), ==, GDK_CONTROL_MASK);