summaryrefslogtreecommitdiff
path: root/ext/smoothstreaming
diff options
context:
space:
mode:
authorThiago Santos <thiago.sousa.santos@collabora.com>2013-04-12 15:55:23 -0300
committerThiago Santos <thiago.sousa.santos@collabora.com>2013-05-07 21:09:48 -0300
commite541ec7e63caf027f03ae42d2e12c41261d140ff (patch)
treeed514f768f530358c8761c84827b7439e9e0ec49 /ext/smoothstreaming
parent6d56eba653e31983f95f079ef3224275b8d3cc3a (diff)
downloadgstreamer-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
Diffstat (limited to 'ext/smoothstreaming')
-rw-r--r--ext/smoothstreaming/gstmssdemux.c13
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;