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 567740 - bogus warning in decodebin2?
bogus warning in decodebin2?
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-01-14 12:20 UTC by Stefan Sauer (gstreamer, gtkdoc dev)
Modified: 2009-04-14 09:36 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Stefan Sauer (gstreamer, gtkdoc dev) 2009-01-14 12:20:26 UTC
GST_DEBUG="*:2" gst-launch-0.10 filesrc location=/home/ensonic/audio/ErsDemo2.mp3 ! decodebin2 ! autoaudiosink
Setting pipeline to PAUSED ...
0:00:00.193331904 18598  0x9bba018 WARN            decodebin2 gstdecodebin2.c:1530:no_more_pads_cb:<decodebin20> We couldn't find a non-completed group !!

isn't that code in decodebin2:no_more_pads_cb() bogus
it warns about something that can't happen
no_more_pads_cb() is specificaly chosen instead of no_more_pads_group_cb() because there is no group, now why does it bother about groups then
Comment 1 Wim Taymans 2009-01-14 12:35:30 UTC
It should not be a warning. I think it's possible that a group is created after the signal is connected in case a demuxer is added to the pipeline later on.
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2009-01-15 14:48:33 UTC
I'll downgrade the warning to info after freeze. I was wondering if the signals for group and non group could happen to be connected for the same elements, but I could not make it happen.
Comment 3 Wim Taymans 2009-04-14 09:36:13 UTC
commit 4265511b7076699d599d70e9cef6a1a1ac912b9b
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Apr 14 11:34:49 2009 +0200

    decodebin2: fix up the debugs and warnings
    
    Use _OBJECT variants because we can. Go over some log statements and put them in
    the right category.
    Fixes #567740.