[gstreamer-bugs] [Bug 630046] sdpdemux: Add optional support for rtspsrc as session element

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=630046
  GStreamer | gst-plugins-bad | 0.10.20

--- Comment #12 from Mark Nauwelaerts <mnauw at users.sourceforge.net> 2010-10-05 10:01:27 UTC ---
Created an attachment (id=171749)
 View: https://bugzilla.gnome.org/attachment.cgi?id=171749
 Review: https://bugzilla.gnome.org/review?bug=630046&attachment=171749

sdpdemux: workaround rtspsrc failing state change

(continued from bug #628214)

The only difference (that matters here) between the original patch and the one
committed is that the original one actually performs _set_state twice.
That (accidentally) happens to workaround rtspsrc failing to reach PLAYING (in
one call).  The attached patch (hackishly) arranges for 2 _set_state calls,
which suffices to eliminate race/deadlock.

A "cleaner fix" may require (tricky?) core modifications, e.g. not having
async_done wipe out the pending state if not a toplevel bin (??)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.