[Bug 786275] adaptivedemux: busy loop when manifest changes liveness

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=786275

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #357546|none                        |reviewed
             status|                            |

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 357546:
 --> (https://bugzilla.gnome.org/review?bug=786275&attachment=357546)

::: 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?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.