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 518033 - Add Monkey's Audio (APE) support
Add Monkey's Audio (APE) support
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal enhancement
: 0.10.4
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-02-22 09:35 UTC by Sebastian Dröge (slomo)
Modified: 2008-04-19 19:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
ffmpeg-ape.diff (3.06 KB, patch)
2008-02-22 09:35 UTC, Sebastian Dröge (slomo)
committed Details | Review

Description Sebastian Dröge (slomo) 2008-02-22 09:35:03 UTC
Hi,
the attached patch adds support for Monkey's Audio (APE) to gst-ffmpeg.

Seems to work fine, except for seeking where I sometimes get an internal data flow error.


Also, the output of ffdemux_ape is not valid APE data anymore, it's stripped from it's headers or something. Do we have a general policy for such things? Maybe a special caps type, i.e. "audio/x-ape-ffdemux" or similar?
Comment 1 Sebastian Dröge (slomo) 2008-02-22 09:35:25 UTC
Created attachment 105749 [details] [review]
ffmpeg-ape.diff
Comment 2 Sebastian Dröge (slomo) 2008-04-19 19:43:36 UTC
Ok, committed this now, I use "audio/x-ffmpeg-parsed-ape" as caps for the crippled data that ffdemux outputs and ffdec wants.

This way we finally get support for Monkey's audio files... :)

2008-04-19  Sebastian Dröge  <slomo@circular-chaos.org>

        * ext/ffmpeg/gstffmpegcodecmap.c: (gst_ffmpeg_codecid_to_caps),
        (gst_ffmpeg_caps_with_codecid), (gst_ffmpeg_formatid_to_caps),
        (gst_ffmpeg_caps_to_codecid), (gst_ffmpeg_get_codecid_longname):
        * ext/ffmpeg/gstffmpegdemux.c: (gst_ffmpegdemux_register):
        Add support for Monkey's Audio (APE). Fixes bug #518033.