From: Tim-Philipp Müller <tim@centricular.net>
Date: Wed, 8 Aug 2012 08:53:26 +0000 (+0100)
Subject: examples: don't put things with side effects inside g_assert()
X-Git-Tag: RELEASE-0.11.93~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c354714544d092b59ea0a76d22b65cc5fd011513;p=platform%2Fupstream%2Fgstreamer.git

examples: don't put things with side effects inside g_assert()

They will be defined away to NOOPs otherwise in release builds.
---

diff --git a/tests/examples/stepping/framestep1.c b/tests/examples/stepping/framestep1.c
index 6eb1881..70a33a9 100644
--- a/tests/examples/stepping/framestep1.c
+++ b/tests/examples/stepping/framestep1.c
@@ -123,8 +123,9 @@ main (int argc, char *argv[])
 
   /* step two frames, flush so that new preroll is queued */
   g_message ("stepping three frames");
-  g_assert (gst_element_send_event (bin,
-          gst_event_new_step (GST_FORMAT_BUFFERS, 2, 1.0, TRUE, FALSE)));
+  if (!gst_element_send_event (bin,
+          gst_event_new_step (GST_FORMAT_BUFFERS, 2, 1.0, TRUE, FALSE)))
+    g_warning ("Filed to send STEP event!");
 
   /* blocks and returns when we received the step done message */
   event_loop (bin);
@@ -138,9 +139,10 @@ main (int argc, char *argv[])
 
   /* step 3 frames, flush so that new preroll is queued */
   g_message ("stepping 120 milliseconds ");
-  g_assert (gst_element_send_event (bin,
+  if (!gst_element_send_event (bin,
           gst_event_new_step (GST_FORMAT_TIME, 120 * GST_MSECOND, 1.0, TRUE,
-              FALSE)));
+              FALSE)))
+    g_warning ("Filed to send STEP event!");
 
   /* blocks and returns when we received the step done message */
   event_loop (bin);