[Bug 720805] pad: stop running tasks before deactivation requests?

[GStreamer (bugzilla.gnome.org)]

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

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |slomo at coaxion.net
         Resolution|                            |NOTABUG

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2013-12-20 10:54:07 UTC ---
1) is what you're supposed to do and what e.g. gst-omx and the androidmedia
plugin are doing. Before chaining up to the parent class' state change function
you must make sure that all pads are unlocked.


3) can't be implemented because pads can't stop their tasks themselves, the
code running inside the tasks is the only one who can unlock and stop

2) is not a solution because it would break ABI. Current subclasses of
GstVideoDecoder assume that all data flow has stopped already when stop() is
called.

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