queue: Don't generate GST_FLOW_ERROR without logging
authorJan Schmidt <jan@centricular.com>
Mon, 2 Jan 2017 15:13:30 +0000 (02:13 +1100)
committerJan Schmidt <jan@centricular.com>
Mon, 2 Jan 2017 15:21:10 +0000 (02:21 +1100)
At least log a message to the debug log when generating
a GST_FLOW_ERROR, to make it possible to find where it came from.

plugins/elements/gstqueue.c

index c6d66eb..57c5759 100644 (file)
@@ -1004,8 +1004,10 @@ gst_queue_handle_sink_event (GstPad * pad, GstObject * parent, GstEvent * event)
       }
       break;
   }
-  if (ret == FALSE)
+  if (ret == FALSE) {
+    GST_ERROR_OBJECT (queue, "Failed to push event");
     return GST_FLOW_ERROR;
+  }
   return GST_FLOW_OK;
 
   /* ERRORS */
@@ -1456,7 +1458,7 @@ next:
   /* ERRORS */
 no_item:
   {
-    GST_CAT_LOG_OBJECT (queue_dataflow, queue,
+    GST_CAT_ERROR_OBJECT (queue_dataflow, queue,
         "exit because we have no item in the queue");
     return GST_FLOW_ERROR;
   }