From: Edward Hervey Date: Mon, 30 Dec 2013 15:47:32 +0000 (+0100) Subject: queue: Remove unneeded checks X-Git-Tag: 1.3.1~254 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=af6fd4f5bb8ad17dfe932ad078c68cb12acee3fc;p=platform%2Fupstream%2Fgstreamer.git queue: Remove unneeded checks item is guaranteed to be non-null. COVERITY CID 1037152 COVERITY CID 1037153 --- diff --git a/plugins/elements/gstqueue.c b/plugins/elements/gstqueue.c index e284570..5bb6774 100644 --- a/plugins/elements/gstqueue.c +++ b/plugins/elements/gstqueue.c @@ -620,6 +620,7 @@ gst_queue_locked_flush (GstQueue * queue, gboolean full) static inline void gst_queue_locked_enqueue_buffer (GstQueue * queue, gpointer item) { + GstQueueItem *qitem; GstBuffer *buffer = GST_BUFFER_CAST (item); gsize bsize = gst_buffer_get_size (buffer); @@ -628,19 +629,18 @@ gst_queue_locked_enqueue_buffer (GstQueue * queue, gpointer item) queue->cur_level.bytes += bsize; apply_buffer (queue, buffer, &queue->sink_segment, TRUE, TRUE); - if (item) { - GstQueueItem *qitem = g_slice_new (GstQueueItem); - qitem->item = item; - qitem->is_query = FALSE; - qitem->size = bsize; - gst_queue_array_push_tail (queue->queue, qitem); - } + qitem = g_slice_new (GstQueueItem); + qitem->item = item; + qitem->is_query = FALSE; + qitem->size = bsize; + gst_queue_array_push_tail (queue->queue, qitem); GST_QUEUE_SIGNAL_ADD (queue); } static inline void gst_queue_locked_enqueue_event (GstQueue * queue, gpointer item) { + GstQueueItem *qitem; GstEvent *event = GST_EVENT_CAST (item); switch (GST_EVENT_TYPE (event)) { @@ -671,12 +671,10 @@ gst_queue_locked_enqueue_event (GstQueue * queue, gpointer item) break; } - if (item) { - GstQueueItem *qitem = g_slice_new (GstQueueItem); - qitem->item = item; - qitem->is_query = FALSE; - gst_queue_array_push_tail (queue->queue, qitem); - } + qitem = g_slice_new (GstQueueItem); + qitem->item = item; + qitem->is_query = FALSE; + gst_queue_array_push_tail (queue->queue, qitem); GST_QUEUE_SIGNAL_ADD (queue); }