[gstreamer-bugs] [Bug 421055] GstGhostPad fails when set to PLAYING for the second time

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=421055

  GStreamer | gstreamer (core) | Ver: HEAD CVS


Christian Schaller (Singer of Songs) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Uraeus at linuxrising.org




------- Comment #2 from Christian Schaller (Singer of Songs)  2007-03-21 15:07 UTC -------
Here is the exact diff causing this:
http://webcvs.freedesktop.org/gstreamer/gstreamer/gst/gstghostpad.c?r1=1.48&r2=1.49

And Wim commit message:
 docs/design/part-gstghostpad.txt:
* gst/gstghostpad.c: (gst_ghost_pad_class_init),
(gst_ghost_pad_internal_do_activate_push),
(gst_ghost_pad_internal_do_activate_pull),
(gst_ghost_pad_do_activate_push), (gst_ghost_pad_do_activate_pull),
(gst_ghost_pad_do_link), (gst_ghost_pad_dispose),
(gst_ghost_pad_new_full), (gst_ghost_pad_set_target):
Do not set the internal pad as a parent anymore so we can avoid
hierarchy linking errors when the ghostpad has no parent yet. This also
fixes failed activation because of unlinked internal pads, which in
turn fixes the impossible case where you have to activate a pad before
you can add it to a running element.
Also fix the docs.
* gst/gstpad.c: (pre_activate), (post_activate),
(gst_pad_set_active), (gst_pad_activate_pull),
(gst_pad_activate_push), (gst_pad_check_pull_range):
Add some more debug info.
Mark activation mode in pre_activate so that we don't try to activate in
endless loops. Fixes #385084.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email