[Bug 786275] adaptivedemux: busy loop when manifest changes liveness
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Mon Aug 14 12:06:01 UTC 2017
https://bugzilla.gnome.org/show_bug.cgi?id=786275
--- Comment #3 from Matthew Waters (ystreet00) <ystreet00 at gmail.com> ---
(In reply to Sebastian Dröge (slomo) from comment #2)
> Review of attachment 357546 [details] [review]:
>
> ::: gst-libs/gst/adaptivedemux/gstadaptivedemux.c
> @@ +4356,3 @@
> + gst_adaptive_demux_stop_manifest_update_task (demux);
> + }
> + gst_adaptive_demux_start_manifest_update_task (demux);
>
> Shouldn't this be in an else case? You now stop the task and always start it
> again?
start has the exact opposite conditions as the above stop internally in the
function.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list