[Bug 648195] outputselector: make it "swap-safe"

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=648195
  GStreamer | gstreamer (core) | git

Håvard Graff (hgr) <havard.graff> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |haakon.sporsheim at gmail.com

--- Comment #3 from Håvard Graff (hgr) <havard.graff at tandberg.com> 2011-04-19 11:16:26 UTC ---
(In reply to comment #2)
> > With this patch, setting active-pad blocks against any activity on that pad.
> 
> That sounds like it might cause problems with backwards-compatibility ..

I thought about that. We did an external trick to make it "half-safe" (pushes
was OK, OOB-events and pad-allocs was not), and that trick could not be used
anymore. Aka, it was not compatible with our own code. 

However, that ment removing a lot of hacks trying to do the job that
outputselector should have done in the first place (IMO).

I think most users of outputselector would feel the same way. It is now
guaranteed to be safe, wheras before it was practically impossible to swap
something out safely.

But of course I am happy to modify the patch if there are any actual
regressions. I'll even write a test-case for it :)

-- 
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.