diff options
author | Thiago Santos <thiago.sousa.santos@collabora.com> | 2013-04-12 15:55:23 -0300 |
---|---|---|
committer | Thiago Santos <thiago.sousa.santos@collabora.com> | 2013-05-07 21:09:48 -0300 |
commit | e541ec7e63caf027f03ae42d2e12c41261d140ff (patch) | |
tree | ed514f768f530358c8761c84827b7439e9e0ec49 | |
parent | 6d56eba653e31983f95f079ef3224275b8d3cc3a (diff) | |
download | gstreamer-plugins-bad-e541ec7e63caf027f03ae42d2e12c41261d140ff.tar.gz |
mssdemux: replace unused parameter
The buffer parameter wasn't being used, it was only to signal if
a buffer was downloaded and advance to the next fragment in the
manifest.
Replace the buffer with a boolean that has the same effect and is
safer
-rw-r--r-- | ext/smoothstreaming/gstmssdemux.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/ext/smoothstreaming/gstmssdemux.c b/ext/smoothstreaming/gstmssdemux.c index 41ab253aa..67a18df7c 100644 --- a/ext/smoothstreaming/gstmssdemux.c +++ b/ext/smoothstreaming/gstmssdemux.c @@ -994,7 +994,7 @@ gst_mss_demux_stream_store_object (GstMssDemuxStream * stream, static GstFlowReturn gst_mss_demux_stream_download_fragment (GstMssDemuxStream * stream, - GstBuffer ** buffer) + gboolean * buffer_downloaded) { GstMssDemux *mssdemux = stream->parent; gchar *path; @@ -1056,8 +1056,8 @@ gst_mss_demux_stream_download_fragment (GstMssDemuxStream * stream, g_object_unref (fragment); - if (buffer) - *buffer = _buffer; + if (buffer_downloaded) + *buffer_downloaded = _buffer != NULL; after_download = g_get_real_time (); if (_buffer) { @@ -1100,7 +1100,7 @@ static void gst_mss_demux_download_loop (GstMssDemuxStream * stream) { GstMssDemux *mssdemux = stream->parent; - GstBuffer *buffer = NULL; + gboolean buffer_downloaded = FALSE; GstFlowReturn ret; GST_LOG_OBJECT (mssdemux, "download loop start %p", stream); @@ -1112,7 +1112,7 @@ gst_mss_demux_download_loop (GstMssDemuxStream * stream) GST_DEBUG_OBJECT (mssdemux, "Finished streams reconfiguration"); GST_OBJECT_UNLOCK (mssdemux); - ret = gst_mss_demux_stream_download_fragment (stream, &buffer); + ret = gst_mss_demux_stream_download_fragment (stream, &buffer_downloaded); if (stream->cancelled) goto cancelled; @@ -1130,9 +1130,10 @@ gst_mss_demux_download_loop (GstMssDemuxStream * stream) stream->download_error_count = 0; - if (buffer) { + if (buffer_downloaded) { gst_mss_stream_advance_fragment (stream->manifest_stream); } + GST_LOG_OBJECT (mssdemux, "download loop end %p", stream); return; |