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 106875 - mpegdemux doesn't support linear PCM streams
mpegdemux doesn't support linear PCM streams
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.6.0
Other All
: Normal enhancement
: 0.6.x
Assigned To: Ronald Bultje
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-02-23 19:50 UTC by Martin Soto
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A tentative patch to fix this problem (2.90 KB, patch)
2003-02-23 19:56 UTC, Martin Soto
none Details | Review
A complete patch to support linear PCM in mpegdemux (7.15 KB, patch)
2003-04-27 23:05 UTC, Martin Soto
none Details | Review

Description Martin Soto 2003-02-23 19:50:08 UTC
The mpegdemux element is not able to extract linear PCM streams from DVD
VOB files.  Linear PCM streams are PES packets in the "private stream 1"
range (id 0xBD, sub id. 0xA0 to 0xA7).  The complete packet format is
described in http://members.freemail.absa.co.za/ginggs/dvd/mpeg2_lpcm.txt.
Comment 1 Martin Soto 2003-02-23 19:56:20 UTC
Created attachment 14554 [details] [review]
A tentative patch to fix this problem
Comment 2 Christian Fredrik Kalager Schaller 2003-02-24 12:16:34 UTC
Since you are calling it tentative I guess Wim should comment on it
before I commit ;) So Wim, what do you say?
Comment 3 Wim Taymans 2003-02-24 20:17:11 UTC
I would suggest to change the mime-type to audio/x-lpcm as audio/pcm
is not registered.
Otherwise, the patch looks good.
Comment 4 Christian Fredrik Kalager Schaller 2003-03-02 09:45:43 UTC
Patch merged into CVS head and Martin added to 0.7 release notes
credits. Thanks for the patch Martin!
Comment 5 Martin Soto 2003-04-27 23:05:08 UTC
Created attachment 16067 [details] [review]
A complete patch to support linear PCM in mpegdemux
Comment 6 Martin Soto 2003-04-27 23:13:01 UTC
I'm reopening this bug since I now have a new bunch of changes.  The
changes in the original patch are not really enough, and the new patch
I just attached, provides the remaining code necessary for full linear
PCM support.  Now, PCM sound will be delivered as raw audio, with
correctly set capabilities.  During the last days, I've been using
this together with audioconvert to play DVDs, it works pretty well.
Comment 7 Christian Fredrik Kalager Schaller 2003-05-12 14:46:22 UTC
Assigning to Ronald for general and 0.6x review.
Comment 8 Ronald Bultje 2003-05-14 21:57:29 UTC
I've committed the patch to HEAD. Thanks for the work!

Re: 0.6.2. That's interesting, but this patch will not work. 0.6.x
doesn't have an audioconvert plugin, so we'd need to do in-plugin byte
swapping. There's probably some more work needed too, plus more
testing. We won't finish that before 0.6.2, sorry. I am keeping it
open for 0.6.3. If we have too much free time we can try getting this
in 0.6.2, but I won't have that time. If anyone else feels like doing
it, please go ahead.
Comment 9 David Schleef 2004-02-11 22:04:45 UTC
This is not going to be backported, as 0.8 is nigh.