After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 116146 - opt scheduler doesn't set event handler on unconnected src pads
opt scheduler doesn't set event handler on unconnected src pads
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
0.6.x
Other Linux
: Normal normal
: 0.6.3
Assigned To: Wim Taymans
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-06-27 20:12 UTC by Thomas Vander Stichele
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (3.96 KB, patch)
2003-06-27 20:44 UTC, Thomas Vander Stichele 		none Details | Review
patch without stupid glist lockup (3.92 KB, patch)
2003-06-28 00:18 UTC, Thomas Vander Stichele 		none Details | Review

Description Thomas Vander Stichele 2003-06-27 20:12:40 UTC 
media-info does metadata querying by using a gnomevfssrc ! decoder pipeline
(ie, src pad of decoder not connected)

looking at optscheduler, it only sets event funcs on linking, so the event
handler never gets set and thus seeking on that pad to a different
logical_stream fails.
Comment 1 Thomas Vander Stichele 2003-06-27 20:44:52 UTC 
Created attachment 17843 [details] [review]
patch
Comment 2 Thomas Vander Stichele 2003-06-27 20:45:43 UTC 
this patch does default eventhandler setting on element addition, then
does it again in link before the switch (to catch pads created after
element addition).

Please tell me what more to change or whether it is good to commit as is.
Comment 3 Thomas Vander Stichele 2003-06-27 21:12:56 UTC 
people tell me I shouldn't be using the event API because it's for
plugins only.

So Company says I should use the to-be-created seek API :)
Comment 4 Thomas Vander Stichele 2003-06-28 00:18:01 UTC 
Created attachment 17845 [details] [review]
patch without stupid glist lockup
Comment 5 Thomas Vander Stichele 2003-07-19 12:31:08 UTC 
wim,
can I get an opinion on this ?

Thanks
Comment 6 Thomas Vander Stichele 2003-08-21 09:18:07 UTC 
pinging
Comment 7 Thomas Vander Stichele 2003-08-26 10:50:44 UTC 
commited to 0.6.2.1 and HEAD, closing.