[Bug 696675] basesink: state change issue when adding alsasink dynamically

[GStreamer (bugzilla.gnome.org)]

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

Wim Taymans <wim.taymans> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |wim.taymans at gmail.com

--- Comment #2 from Wim Taymans <wim.taymans at gmail.com> 2013-04-03 10:33:38 UTC ---
It does indeed not call the state_change function and it should do that.

I think your patch is however racy. It is possible that while you call the
gst_element_change_state() a concurrent state change might happen. I think it
somehow needs to take the STATE_LOCK as is a requirement for
gst_element_change_state(). I will have a deeper look because this code should
theoretically be similar to how the bin continues its state.

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