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 414887 - [gconf] gconfaudiosink doesn't set GST_ELEMENT_IS_SINK until READY
[gconf] gconfaudiosink doesn't set GST_ELEMENT_IS_SINK until READY
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 0.10.6
Assigned To: Wim Taymans
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-03-05 12:58 UTC by Jonathan Matthew
Modified: 2007-03-07 11:15 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Jonathan Matthew 2007-03-05 12:58:30 UTC
If you set up a pipeline with a gconfaudiosink as its only sink without first changing the sink's state to READY, the pipeline thinks it doesn't have sink, so position queries fail.  Other audio sinks (and autoaudiosink) work properly.

This can be reproduced with rhythmbox with the visualisation plugin disabled - the song position display will stay at 0:00.  Totem doesn't exhibit the problem as it sets the sink to READY before adding it to the pipeline.

This problem does not occur with -good 0.10.5.
Comment 1 Wim Taymans 2007-03-05 14:46:49 UTC
        * ext/gconf/gstswitchsink.c: (gst_switch_sink_reset):
        Mark us as a sink when we have no fakesink in NULL. Fixes #414887.
Comment 2 Jan Schmidt 2007-03-05 15:56:45 UTC
New, more different fix:

        * ext/gconf/gstswitchsink.c: (gst_switch_sink_reset),
        (gst_switch_commit_new_kid), (gst_switch_sink_set_child):
        Install fakesink in NULL by fixing some broken logic. This obviates
        the need to manually set _IS_SINK.
        Add some comments and remove a little cruft while I'm at it.