tests: Address unused but set variables
authorRobert Swain <robert.swain@collabora.co.uk>
Fri, 15 Apr 2011 13:11:35 +0000 (15:11 +0200)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Sat, 16 Apr 2011 12:10:58 +0000 (13:10 +0100)
GCC 4.6.x spits warnings about such usage of variables.

tests/check/elements/deinterlace.c
tests/check/elements/rtp-payloading.c
tests/check/pipelines/flacdec.c
tests/examples/level/level-example.c
tests/icles/videocrop-test.c
tests/icles/ximagesrc-test.c

index b111c86f7414757eed3e159987c58765568b77cc..6aeb3def81553408896d4e631f9100f93410d1f1 100644 (file)
@@ -28,7 +28,7 @@
 static gboolean
 gst_caps_is_interlaced (GstCaps * caps)
 {
-  GstStructure *structure;
+  GstStructure G_GNUC_UNUSED *structure;
   gboolean interlaced = FALSE;
 
   fail_unless (gst_caps_is_fixed (caps));
index 245f3321999d9b310d0cd834ea245b3ddd9ae45b..b2160f4a585f317c928efe9e8ccfe126c0ef37c7 100644 (file)
@@ -291,13 +291,12 @@ rtp_pipeline_run (rtp_pipeline * p)
   for (i = 0; i < LOOP_COUNT; i++) {
     const char *frame_data_pointer = p->frame_data;
     int res;
-
     int frame_count = p->frame_count;
 
     /* Write in to the pipe. */
     while (frame_count > 0) {
       res = write (p->fd[1], frame_data_pointer, p->frame_data_size);
-
+      fail_unless_equals_int (res, p->frame_data_size);
       frame_data_pointer += p->frame_data_size;
       frame_count--;
     }
index abf9a058ce969318ba9289a52222ffee525fcd3a..ce8546e9952dbd5e0ff27f5846f08cbb1f6f9c54 100644 (file)
@@ -110,7 +110,6 @@ GST_START_TEST (test_decode_seek_full)
   GstEvent *event;
   GstBuffer *buffer = NULL;
   guint16 first_sample = 0;
-  gboolean result;
   guint size = 0;
   gchar *path =
       g_build_filename (GST_TEST_FILES_PATH, "audiotestsrc.flac", NULL);
@@ -133,7 +132,7 @@ GST_START_TEST (test_decode_seek_full)
   /* do a seek that should give us the complete output */
   event = gst_event_new_seek (1.0, GST_FORMAT_DEFAULT, GST_SEEK_FLAG_FLUSH,
       GST_SEEK_TYPE_SET, 0, GST_SEEK_TYPE_SET, 20480);
-  result = gst_element_send_event (appsink, event);
+  fail_unless (gst_element_send_event (appsink, event));
 
   gst_element_set_state (pipeline, GST_STATE_PLAYING);
 
@@ -169,7 +168,6 @@ GST_START_TEST (test_decode_seek_partial)
   GstElement *appsink;
   GstEvent *event;
   GstBuffer *buffer = NULL;
-  gboolean result;
   guint size = 0;
   guint16 first_sample = 0;
   gchar *path =
@@ -194,7 +192,7 @@ GST_START_TEST (test_decode_seek_partial)
   event = gst_event_new_seek (1.0, GST_FORMAT_DEFAULT, GST_SEEK_FLAG_FLUSH,
       GST_SEEK_TYPE_SET, 0, GST_SEEK_TYPE_SET, 1024);
   GST_DEBUG ("seeking");
-  result = gst_element_send_event (appsink, event);
+  fail_unless (gst_element_send_event (appsink, event));
   GST_DEBUG ("seeked");
 
   gst_element_set_state (pipeline, GST_STATE_PLAYING);
index ece3647141631fa77cfa33e822cca75ef4402942..aa8b416f4844c31cea90a0cff7b9741abb041ba4 100644 (file)
@@ -82,7 +82,7 @@ main (int argc, char *argv[])
   GstElement *pipeline;
   GstCaps *caps;
   GstBus *bus;
-  gint watch_id;
+  guint watch_id;
   GMainLoop *loop;
 
   gst_init (&argc, &argv);
@@ -120,5 +120,7 @@ main (int argc, char *argv[])
   loop = g_main_loop_new (NULL, FALSE);
   g_main_loop_run (loop);
 
+  g_source_remove (watch_id);
+  g_main_loop_unref (loop);
   return 0;
 }
index 94656342df2d4586d78e030368c77c485521a927..a4874e160a86f79b23e0b452893a682121a47764 100644 (file)
@@ -190,7 +190,6 @@ main (int argc, char **argv)
   GError *opt_err = NULL;
 
   GstElement *pipeline, *src, *filter1, *crop, *scale, *filter2, *csp, *sink;
-  GMainLoop *loop;
   GstCaps *filter_caps = NULL;
   GList *caps_list, *l;
 
@@ -209,8 +208,6 @@ main (int argc, char **argv)
 
   GST_DEBUG_CATEGORY_INIT (videocrop_test_debug, "videocroptest", 0, "vctest");
 
-  loop = g_main_loop_new (NULL, FALSE);
-
   pipeline = gst_pipeline_new ("pipeline");
   src = gst_element_factory_make ("videotestsrc", "videotestsrc");
   g_assert (src != NULL);
index f51913a993f3f432b2482bec1d18afad0a3e6b4b..a01bf656b3a8de321de7ac1dd5aab99b30fdd531 100644 (file)
@@ -37,7 +37,6 @@ int
 main (int argc, char **argv)
 {
   GstElement *pipeline;
-  GstBus *bus;
 #ifndef G_DISABLE_ASSERT
   GstState state, pending;
 #endif
@@ -53,8 +52,6 @@ main (int argc, char **argv)
 
   loop = g_main_loop_new (NULL, FALSE);
 
-  bus = gst_pipeline_get_bus (GST_PIPELINE (pipeline));
-
   gst_element_set_state (pipeline, GST_STATE_PLAYING);
 
   /* lets check it gets to PLAYING */