[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