From a01133dd8f00a8563ba41b0c7778ccbda11b234e Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Wed, 8 Jul 2015 11:32:58 +0100 Subject: [PATCH] dashdemux: remove redundant else statement Else statement is redundant when the if block finishes with an return. --- ext/dash/gstdashdemux.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/ext/dash/gstdashdemux.c b/ext/dash/gstdashdemux.c index e40af5a942..9b91e06e22 100644 --- a/ext/dash/gstdashdemux.c +++ b/ext/dash/gstdashdemux.c @@ -1334,15 +1334,14 @@ gst_dash_demux_stream_fragment_finished (GstAdaptiveDemux * demux, GstDashDemuxStream *dashstream = (GstDashDemuxStream *) stream; if (gst_mpd_client_has_isoff_ondemand_profile (dashdemux->client) && - dashstream->sidx_parser.status == GST_ISOFF_SIDX_PARSER_FINISHED) { + dashstream->sidx_parser.status == GST_ISOFF_SIDX_PARSER_FINISHED) /* fragment is advanced on data_received when byte limits are reached */ return GST_FLOW_OK; - } else { - if (G_UNLIKELY (stream->downloading_header || stream->downloading_index)) - return GST_FLOW_OK; - return gst_adaptive_demux_stream_advance_fragment (demux, stream, - stream->fragment.duration); - } + if (G_UNLIKELY (stream->downloading_header || stream->downloading_index)) + return GST_FLOW_OK; + + return gst_adaptive_demux_stream_advance_fragment (demux, stream, + stream->fragment.duration); } static GstFlowReturn