mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-12-23 00:36:51 +00:00
Code Issues 1 Projects Releases Wiki Activity

adaptivedemux2: Add state checks and clean up obsolete variables

Browse source 
The cancelled flag was only set in the stream finalize()
method, after all activity on the stream has stopped anyway.
Replace uses of cancelled with checks on the stream state.

Remove the replaced flag, which was checked but never set
to TRUE anywhere any more.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
This commit is contained in:
Jan Schmidt 2022-08-16 23:01:46 +10:00
parent 730ed3a69a
commit 93f80bb061
2 changed files with 20 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
subprojects/gst-plugins-good/ext/adaptivedemux2/gstadaptivedemux-stream.c
View file

@ -85,7 +85,6 @@ gst_adaptive_demux2_stream_finalize (GObject * object)
if (stream->download_request)
download_request_unref (stream->download_request);
stream->cancelled = TRUE;
g_clear_error (&stream->last_error);
gst_adaptive_demux2_stream_fragment_clear (&stream->fragment);
@ -697,7 +696,7 @@ gst_adaptive_demux2_stream_push_buffer (GstAdaptiveDemux2Stream * stream,
gst_pad_send_event (stream->parsebin_sink, buffer_gap);
}
if (G_UNLIKELY (stream->cancelled)) {
if (G_UNLIKELY (stream->state == GST_ADAPTIVE_DEMUX2_STREAM_STATE_STOPPED)) {
GST_LOG_OBJECT (demux, "Stream was cancelled");
return GST_FLOW_FLUSHING;
}
@ -716,7 +715,8 @@ gst_adaptive_demux2_stream_parse_buffer (GstAdaptiveDemux2Stream * stream,
GstFlowReturn ret = GST_FLOW_OK;
/* do not make any changes if the stream is cancelled */
if (G_UNLIKELY (stream->cancelled)) {
if (G_UNLIKELY (stream->state == GST_ADAPTIVE_DEMUX2_STREAM_STATE_STOPPED)) {
GST_DEBUG_OBJECT (stream, "Stream was stopped. Aborting");
gst_buffer_unref (buffer);
return GST_FLOW_FLUSHING;
}
@ -746,7 +746,8 @@ gst_adaptive_demux2_stream_parse_buffer (GstAdaptiveDemux2Stream * stream,
if (ret == GST_FLOW_FLUSHING) {
/* do not make any changes if the stream is cancelled */
if (G_UNLIKELY (stream->cancelled)) {
if (stream->state == GST_ADAPTIVE_DEMUX2_STREAM_STATE_STOPPED) {
GST_DEBUG_OBJECT (stream, "Stream was stopped. Aborting");
return ret;
}
}
@ -1158,6 +1159,12 @@ on_download_error (DownloadRequest * request, DownloadRequestState state,
guint last_status_code = request->status_code;
gboolean live;
if (stream->state != GST_ADAPTIVE_DEMUX2_STREAM_STATE_DOWNLOADING) {
GST_DEBUG_OBJECT (stream, "Stream state changed to %d. Aborting",
stream->state);
return;
}
stream->download_active = FALSE;
stream->last_status_code = last_status_code;
@ -1336,8 +1343,11 @@ on_download_complete (DownloadRequest * request, DownloadRequestState state,
stream->download_active = FALSE;
if (G_UNLIKELY (stream->cancelled))
if (stream->state != GST_ADAPTIVE_DEMUX2_STREAM_STATE_DOWNLOADING) {
GST_DEBUG_OBJECT (stream, "Stream state changed to %d. Aborting",
stream->state);
return;
}
GST_DEBUG_OBJECT (stream,
"Stream %p %s download for %s is complete with state %d",
@ -1726,9 +1736,6 @@ gst_adaptive_demux2_stream_handle_playlist_eos (GstAdaptiveDemux2Stream *
gst_adaptive_demux2_stream_wants_manifest_update (demux);
return;
}
if (stream->replaced)
return;
}
gst_adaptive_demux2_stream_end_of_manifest (stream);
@ -1766,6 +1773,9 @@ gst_adaptive_demux2_stream_load_a_fragment (GstAdaptiveDemux2Stream * stream)
GST_ERROR_OBJECT (stream,
"Unexpected stream state EOS. The stream should not be running now.");
return FALSE;
case GST_ADAPTIVE_DEMUX2_STREAM_STATE_STOPPED:
/* The stream was stopped. Just finish up */
return FALSE;
default:
GST_ERROR_OBJECT (stream, "Unexpected stream state %d", stream->state);
g_assert_not_reached ();
@ -1838,9 +1848,8 @@ gst_adaptive_demux2_stream_load_a_fragment (GstAdaptiveDemux2Stream * stream)
case GST_FLOW_FLUSHING:
/* Flushing is normal, the target track might have been unselected */
if (G_UNLIKELY (stream->cancelled))
return FALSE;
GST_DEBUG_OBJECT (stream, "Got flushing return. Stopping callback.");
return FALSE;
default:
if (ret <= GST_FLOW_ERROR) {
GST_WARNING_OBJECT (demux, "Error while downloading fragment");
@ -2004,8 +2013,6 @@ gst_adaptive_demux2_stream_start (GstAdaptiveDemux2Stream * stream)
stream->state == GST_ADAPTIVE_DEMUX2_STREAM_STATE_RESTART) {
GST_LOG_OBJECT (stream, "Activating stream. Current state %d",
stream->state);
stream->cancelled = FALSE;
stream->replaced = FALSE;
stream->last_ret = GST_FLOW_OK;
if (stream->state == GST_ADAPTIVE_DEMUX2_STREAM_STATE_STOPPED)

2
subprojects/gst-plugins-good/ext/adaptivedemux2/gstadaptivedemux.h
View file

@ -329,8 +329,6 @@ struct _GstAdaptiveDemux2Stream
gboolean pending_tracks; /* if we need to discover tracks dynamically for this stream */
gboolean download_finished;
gboolean cancelled;
gboolean replaced; /* replaced in a bitrate switch (used with cancelled) */
gboolean starting_fragment;
gboolean first_fragment_buffer;