[Bug 755169] dashdemux: can we have multiple seek events at the same time?

[GStreamer (GNOME Bugzilla)]

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

--- Comment #26 from Florin Apostol <florin.apostol at oregan.net> ---
(In reply to Vincent Penquerc'h from comment #25)

> I'm not sure adding yet another lock is a good idea though. I understand
> your idea, but I'm concerned it will make future changes even harder.
> 
We need to clarify the use of mutexes. For each shared variable (stream, demux)
we should document what mutex protects it. Because now we protect the
demux->cancelled sometimes with GST_OBJECT_LOCK (demux) sometimes with
GST_MANIFEST_LOCK (demux). And this is not good.

> What do you think about the patch below (on top of my last one here), 

Can you provide a set of patches that should be applied on mainline (or push
your tree somewhere I can see it? It is hard for me to figure out the complete
solution you propose. For example do you still intend to keep the copy of the
streams list? If yes, why? Because I think we should not need it and it
complicates things very much.

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