qtdemux: Remove useless return variable
authorSeungha Yang <sh.yang@lge.com>
Fri, 18 Nov 2016 13:45:45 +0000 (22:45 +0900)
committerSebastian Dröge <sebastian@centricular.com>
Fri, 18 Nov 2016 16:05:23 +0000 (18:05 +0200)
qtdemux_expose_streams() returns flow error immediately, if there is an error.
So, the variable for the flow return is not needed.

https://bugzilla.gnome.org/show_bug.cgi?id=774674

gst/isomp4/qtdemux.c

index c61a586..b66c34c 100644 (file)
@@ -11377,13 +11377,12 @@ static GstFlowReturn
 qtdemux_expose_streams (GstQTDemux * qtdemux)
 {
   gint i;
-  GstFlowReturn ret = GST_FLOW_OK;
   GSList *oldpads = NULL;
   GSList *iter;
 
   GST_DEBUG_OBJECT (qtdemux, "exposing streams");
 
-  for (i = 0; ret == GST_FLOW_OK && i < qtdemux->n_streams; i++) {
+  for (i = 0; i < qtdemux->n_streams; i++) {
     QtDemuxStream *stream = qtdemux->streams[i];
     GstPad *oldpad = stream->pad;
     GstTagList *list;
@@ -11450,7 +11449,7 @@ qtdemux_expose_streams (GstQTDemux * qtdemux)
   }
 
   qtdemux->exposed = TRUE;
-  return ret;
+  return GST_FLOW_OK;
 }
 
 /* check if major or compatible brand is 3GP */