[Bug 773341] urisourcebin: Cleanup unused output slot

[GStreamer (GNOME Bugzilla)]

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

--- Comment #7 from Seungha Yang <sh.yang at lge.com> ---
(In reply to Sebastian Dröge (slomo) from comment #6)
> Review of attachment 338244 [details] [review]:
> 

Thanks for very quick review !! :)

> Always having yet another thread when having a demuxer seems a bit heavy.
> Maybe you can make use of gst_element_call_async() here (in a one-shot way),
> which uses a thread pool?

I'll try to study more about "gst_element_call_async()" Thanks.

> @@ +1864,3 @@
> +  /* run drained_task () */
> +  urisrc->drained_task =
> +      gst_task_new ((GstTaskFunction) cleanup_slot_loop, urisrc, NULL);
> 
> There can also be multiple demuxers, e.g. AVI containing DV, MXF containing
> MPEG-PS/TS, etc

Could you kindly explain more about "AVI containing DV, MXF containing
MPEG-PS/TS"? The "demux" in urisourcebin seems to be only AdaptiveDemux for
DASH/HLS/Smooth streaming. Are there any case that those multiple adaptive
demuxers exist at one time?

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