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 135862 - [wavparse] add ADPCM support
[wavparse] add ADPCM support
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.7.4
Other Linux
: Normal normal
: 0.8.2
Assigned To: Ronald Bultje
GStreamer Maintainers
Depends on:
Blocks: 138435
 
 
Reported: 2004-03-01 19:16 UTC by Frederic Crozat
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch from ronald to make wavparse handle adpcm (1.67 KB, patch)
2004-04-15 09:49 UTC, Thomas Vander Stichele
none Details | Review

Description Frederic Crozat 2004-03-01 19:16:58 UTC
With either gstreamer 0.6.4 or 0.7.4, try to use
gst-launch-ext on test file from bug
http://bugzilla.gnome.org/show_bug.cgi?id=107483 (it is a MS ADPCM wave file)

You'll get a :
ERROR: /pipeline0/wavparse0: wavparse: format 2 not handled

Trying to use audiofile source is also problematic.
Comment 1 David Schleef 2004-03-30 22:41:51 UTC
Not playing in gst-launch-ext isn't really a bug.

However, it doesn't really play well with gst-player, either.
Comment 2 Thomas Vander Stichele 2004-04-15 09:49:25 UTC
Created attachment 26680 [details] [review]
patch from ronald to make wavparse handle adpcm

the attached patch from Ronald fixes wavparse.
However, it doesn't help the situation, since spider still plugs wavparse
directly connected to osssink, while it should insert an ffmpeg decoder for the
format.  This is probably because spider sees one of the pad templates of wav
parse as directly connectable to osssink, but on actual caps nego wavparse sets
the adpcm mime type explicitly causing it to fail.

So not putting this patch in 0.8.1 until that is resolved.
Comment 3 Ronald Bultje 2004-04-16 01:23:55 UTC
Committed, so the remaining thing is to fix spider.
Comment 4 Ronald Bultje 2004-04-18 23:39:43 UTC
Fixed in CVS by making it a sometimes pad. Workaround can be reverted once
spider or another autoplugger is fixed.
Comment 5 Johan (not receiving bugmail) Dahlin 2004-04-19 08:56:53 UTC
Ronald, can you please add a comment in the source with that and a reference to
this bug?
Comment 6 Ronald Bultje 2004-04-19 12:07:36 UTC
There is. gst-plugins/gst/wavparse/gstwavparse.c:66

GST_PAD_SOMETIMES, /* FIXME: spider */

Benjamin and me know what that means.