[gstreamer-bugs] [Bug 316856] GstBin does state changes in wrong order sometimes

[GStreamer (bugzilla.gnome.org) <bug...@bu...>]

Do not reply to this email.  You can add comments to this bug at
 http://bugzilla.gnome.org/show_bug.cgi?id=3D316856
 GStreamer | gstreamer (core) | Ver: HEAD CVS

Tim-Philipp M=FCller changed:

           What    |Removed                     |Added
-------------------------------------------------------------------------=
---
  Attachment #52469|                            |committed
             Status|                            |



------- Additional Comments From Tim-Philipp M=FCller  2005-09-23 17:38 -=
------
2005-09-23  Tim-Philipp Muller  <tim at centricular dot net>

        * check/gst/gstbin.c: (test_children_state_change_order_flagged_s=
ink),
        (test_children_state_change_order_semi_sink), (gst_bin_suite):
          Added test to check state change order in bins (can still be ma=
de
          to fail here under heavy disk load; bails out with 'Push on pad
          fakesink:sink0, but it was not activated in push mode').

        * gst/gstbin.c: (gst_bin_class_init), (gst_bin_change_state):
          Fix state change order when there is only a semi sink (#316856)

        * gst/gstbus.c: (gst_bus_class_init):
          Use _class_peek_parent(), not _class_ref(); fix docs to say
          'default main context' instead of 'mainloop' where that is
          what's meant.

        * gst/gstelement.c: (gst_element_commit_state),
        (gst_element_set_state):
          Fix typos in debug messages


Committed patch and test case. Test case generally works fine but can be =
made to
fail with an=20

   Push on pad fakesink:sink0, but it was not activated in push mode

assertion/warning under heavy disk load when run in a loop. Dunno what th=
at is
or how to fix it.

Cheers
 -Tim
=20

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