summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNicolas Dufresne <nicolas.dufresne@collabora.com>2018-03-07 11:16:00 -0500
committerNicolas Dufresne <nicolas.dufresne@collabora.com>2018-03-07 20:03:56 -0500
commit12c5d903c9cf80d9365d57b4643fef9f57aa4d60 (patch)
tree870c8ddad4cf1072cacd712cc9cc84be73f30220
parentf204b57fa54447d7d0052cd96f0fc5cb76016af6 (diff)
basesrc: No need to stop flushing in start_complete
The flushing state is handled a bit differently, there is no need to stop flushing in start_complete. This would other result in unlock_stop being called without unlock_start. Unlike what the old comment says, there is no need to take the live lock here, we are still single threaded at this point (app thread or the state change thread). Also, we will wait for playing state in create/getrange, no need to do that twice. https://bugzilla.gnome.org/show_bug.cgi?id=794149
-rw-r--r--libs/gst/base/gstbasesrc.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/libs/gst/base/gstbasesrc.c b/libs/gst/base/gstbasesrc.c
index da73c8c05..f85f32105 100644
--- a/libs/gst/base/gstbasesrc.c
+++ b/libs/gst/base/gstbasesrc.c
@@ -3512,10 +3512,6 @@ gst_base_src_start_complete (GstBaseSrc * basesrc, GstFlowReturn ret)
GST_DEBUG_OBJECT (basesrc, "is random_access: %d", basesrc->random_access);
- /* stop flushing now but for live sources, still block in the LIVE lock when
- * we are not yet PLAYING */
- gst_base_src_set_flushing (basesrc, FALSE);
-
gst_pad_mark_reconfigure (GST_BASE_SRC_PAD (basesrc));
GST_OBJECT_LOCK (basesrc->srcpad);