[gst-cvs] gstreamer: elements: Stop using GST_FLOW_IS_FATAL()

Sebastian Dröge slomo at kemper.freedesktop.org
Fri Aug 27 10:02:49 PDT 2010


Module: gstreamer
Branch: master
Commit: 2e6f80bf8120628be1f8a7d0181ba3999689a251
URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=2e6f80bf8120628be1f8a7d0181ba3999689a251

Author: Sebastian Dröge <sebastian.droege at collabora.co.uk>
Date:   Thu Aug 26 23:39:06 2010 +0200

elements: Stop using GST_FLOW_IS_FATAL()

---

 plugins/elements/gstmultiqueue.c |    2 +-
 plugins/elements/gstqueue.c      |    3 +--
 plugins/elements/gstqueue2.c     |    7 +++----
 3 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/plugins/elements/gstmultiqueue.c b/plugins/elements/gstmultiqueue.c
index b9fa663..d9bbba9 100644
--- a/plugins/elements/gstmultiqueue.c
+++ b/plugins/elements/gstmultiqueue.c
@@ -1123,7 +1123,7 @@ out_flushing:
     /* upstream needs to see fatal result ASAP to shut things down,
      * but might be stuck in one of our other full queues;
      * so empty this one and trigger dynamic queue growth */
-    if (GST_FLOW_IS_FATAL (sq->srcresult)) {
+    if (sq->srcresult <= GST_FLOW_UNEXPECTED) {
       gst_data_queue_flush (sq->queue);
       single_queue_underrun_cb (sq->queue, sq);
     }
diff --git a/plugins/elements/gstqueue.c b/plugins/elements/gstqueue.c
index 0306b04..34d08ed 100644
--- a/plugins/elements/gstqueue.c
+++ b/plugins/elements/gstqueue.c
@@ -1234,8 +1234,7 @@ out_flushing:
     GST_QUEUE_MUTEX_UNLOCK (queue);
     /* let app know about us giving up if upstream is not expected to do so */
     /* UNEXPECTED is already taken care of elsewhere */
-    if (eos && (GST_FLOW_IS_FATAL (ret) || ret == GST_FLOW_NOT_LINKED) &&
-        (ret != GST_FLOW_UNEXPECTED)) {
+    if (eos && (ret == GST_FLOW_NOT_LINKED || ret < GST_FLOW_UNEXPECTED)) {
       GST_ELEMENT_ERROR (queue, STREAM, FAILED,
           (_("Internal data flow error.")),
           ("streaming task paused, reason %s (%d)",
diff --git a/plugins/elements/gstqueue2.c b/plugins/elements/gstqueue2.c
index c56f887..bb09e5a 100644
--- a/plugins/elements/gstqueue2.c
+++ b/plugins/elements/gstqueue2.c
@@ -2303,8 +2303,7 @@ out_flushing:
     GST_QUEUE2_MUTEX_UNLOCK (queue);
     /* let app know about us giving up if upstream is not expected to do so */
     /* UNEXPECTED is already taken care of elsewhere */
-    if (eos && (GST_FLOW_IS_FATAL (ret) || ret == GST_FLOW_NOT_LINKED) &&
-        (ret != GST_FLOW_UNEXPECTED)) {
+    if (eos && (ret == GST_FLOW_NOT_LINKED || ret < GST_FLOW_UNEXPECTED)) {
       GST_ELEMENT_ERROR (queue, STREAM, FAILED,
           (_("Internal data flow error.")),
           ("streaming task paused, reason %s (%d)",
@@ -2684,7 +2683,7 @@ gst_queue2_src_activate_pull (GstPad * pad, gboolean active)
         result = gst_queue2_open_temp_location_file (queue);
       } else if (!queue->ring_buffer) {
         queue->ring_buffer = g_malloc (queue->ring_buffer_max_size);
-        result = !!queue->ring_buffer;
+        result = ! !queue->ring_buffer;
       } else {
         result = TRUE;
       }
@@ -2887,7 +2886,7 @@ gst_queue2_set_property (GObject * object,
       break;
     case PROP_RING_BUFFER_MAX_SIZE:
       queue->ring_buffer_max_size = g_value_get_uint64 (value);
-      queue->use_ring_buffer = !!queue->ring_buffer_max_size;
+      queue->use_ring_buffer = ! !queue->ring_buffer_max_size;
       break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);





More information about the Gstreamer-commits mailing list