navigation/v4l: Don't use g_return_val_if_fail for computed/used values.
authorEdward Hervey <bilboed@bilboed.com>
Sat, 4 Apr 2009 14:28:14 +0000 (16:28 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Sat, 4 Apr 2009 14:28:14 +0000 (16:28 +0200)
gst-libs/gst/interfaces/navigation.c
sys/v4l/v4lsrc_calls.c

index 892ee75..8aa781d 100644 (file)
@@ -520,13 +520,13 @@ gboolean
 gst_navigation_message_parse_mouse_over (GstMessage * message,
     gboolean * active)
 {
-  g_return_val_if_fail (GST_NAVIGATION_MESSAGE_HAS_TYPE (message, MOUSE_OVER),
-      FALSE);
+  if (!GST_NAVIGATION_MESSAGE_HAS_TYPE (message, MOUSE_OVER))
+    return FALSE;
 
   if (active) {
     const GstStructure *s = gst_message_get_structure (message);
-    g_return_val_if_fail (gst_structure_get_boolean (s, "active", active),
-        FALSE);
+    if (gst_structure_get_boolean (s, "active", active) == FALSE)
+      return FALSE;
   }
 
   return TRUE;
index 9ba2558..7a32099 100644 (file)
@@ -707,7 +707,8 @@ gst_v4lsrc_buffer_new (GstV4lSrc * v4lsrc, gint num)
 
   GST_DEBUG_OBJECT (v4lsrc, "creating buffer for frame %d", num);
 
-  g_return_val_if_fail (gst_v4lsrc_get_fps (v4lsrc, &fps_n, &fps_d), NULL);
+  if (!(gst_v4lsrc_get_fps (v4lsrc, &fps_n, &fps_d)))
+    return NULL;
 
   buf = (GstBuffer *) gst_mini_object_new (GST_TYPE_V4LSRC_BUFFER);