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 325652 - many plugins don't pass unhandled queries upstream
many plugins don't pass unhandled queries upstream
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
0.10.0
Other All
: Normal normal
: 0.10.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-01-03 18:36 UTC by Philippe Rouquier
Modified: 2006-01-10 11:53 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
This patch should fix this kind of problem for all plugins in gst-plugins-ugly (3.12 KB, patch)
2006-01-04 12:55 UTC, Philippe Rouquier
committed Details | Review

Description Philippe Rouquier 2006-01-03 18:36:10 UTC
Please describe the problem:
gstmad doesn't pass the query upstream as it should. (At least in playbin!)

Steps to reproduce:



Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Philippe Rouquier 2006-01-03 18:56:00 UTC
The same can be said of gstid3tag. It doesn't pass query upstream.
Comment 2 Philippe Rouquier 2006-01-04 12:55:46 UTC
Created attachment 56771 [details] [review]
This patch should fix this kind of problem for all plugins in gst-plugins-ugly
Comment 3 Tim-Philipp Müller 2006-01-10 11:53:38 UTC
Thanks, fixed in CVS:

2006-01-10  Tim-Philipp Müller  <tim at centricular dot net>

        * ext/amrnb/amrnbparse.c: (gst_amrnbparse_query):
        * ext/mad/gstid3tag.c: (gst_id3_tag_src_query):
        * ext/mad/gstmad.c: (gst_mad_src_query):
        * ext/mpeg2dec/gstmpeg2dec.c: (gst_mpeg2dec_src_query):
        * ext/sidplay/gstsiddec.cc:
        * gst/mpegstream/gstmpegparse.c: (gst_mpeg_parse_handle_src_query):
        * gst/realmedia/rmdemux.c: (gst_rmdemux_src_query):
          Pass unhandled queries upstream (useful e.g. for SEEKING query)
          (fixes #325652; based on patch by: Philippe); make rmdemux return
          FALSE for position queries, instead of setting -1 as value and
          returning TRUE.

Btw, please add a surname to your real name in your bugzilla account so that we can attribute patches to you properly (also makes it easier to tell you apart from other Philippes) :)