[FFmpeg-devel] [PATCH 0/3] aiff: add support for XA ADPCM
misty at brew.sh
misty at brew.sh
Wed Jan 3 10:14:21 EET 2018
From: Misty De Meo <mistydemeo at gmail.com>
This patchset adds support for the `APCM` chunk type in AIFF,
which is used to store CD-ROM XA ADPCM audio. The only application
I currently know of which uses this is the Sega Saturn game
Sonic Jam, but there might be others. I've included a new fate test
for this; here's the media file it uses:
https://public.drac.at/S1_93.ADP
I've also fixed a minor bug that this exposed in the XA decoder.
If the frame size isn't a multiple of 128 bytes, the decoder would
leave the remainder unconsumed and it would be passed to the next
adpcm_decode_frame() call. This most likely didn't come up before
because the most common container (psxstr) has a frame size that's
an exact multiple of 128. AIFF APCM files, however, use a frame size
of 2324 bytes - the size of a mode 2 form 2 sector. That leaves a
remainder of 20 padding bytes.
Misty De Meo (3):
aiff: add support for XA ADPCM
adpcm: consume remainder after consuming XA chunks
fate: add xa-aiff test
libavcodec/adpcm.c | 7 ++++++
libavformat/aiffdec.c | 7 ++++++
tests/fate/adpcm.mak | 3 +++
tests/ref/fate/adpcm-xa-aiff | 51 ++++++++++++++++++++++++++++++++++++++++++++
4 files changed, 68 insertions(+)
create mode 100644 tests/ref/fate/adpcm-xa-aiff
--
2.15.1
More information about the ffmpeg-devel
mailing list