[FFmpeg-devel] [PATCH 1/2] avcodec: add decoder for Rayman 2's ADPCM variant
Paul B Mahol
onemda at gmail.com
Mon Feb 17 11:20:59 EET 2020
On 2/17/20, Zane van Iperen <zane at zanevaniperen.com> wrote:
> Adds support for the ADPCM variant used in Rayman 2's files.
>
> Signed-off-by: Zane van Iperen <zane at zanevaniperen.com>
> ---
> libavcodec/Makefile | 1 +
> libavcodec/adpcm.c | 24 ++++++++++++++++++++++++
> libavcodec/allcodecs.c | 1 +
> libavcodec/avcodec.h | 1 +
> libavcodec/codec_desc.c | 7 +++++++
> libavcodec/version.h | 2 +-
> 6 files changed, 35 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/Makefile b/libavcodec/Makefile
> index 46da42570f..f2facab652 100644
> --- a/libavcodec/Makefile
> +++ b/libavcodec/Makefile
> @@ -834,6 +834,7 @@ OBJS-$(CONFIG_ADPCM_G726LE_DECODER) += g726.o
> OBJS-$(CONFIG_ADPCM_G726LE_ENCODER) += g726.o
> OBJS-$(CONFIG_ADPCM_IMA_AMV_DECODER) += adpcm.o adpcm_data.o
> OBJS-$(CONFIG_ADPCM_IMA_APC_DECODER) += adpcm.o adpcm_data.o
> +OBJS-$(CONFIG_ADPCM_IMA_APM_DECODER) += adpcm.o adpcm_data.o
> OBJS-$(CONFIG_ADPCM_IMA_DAT4_DECODER) += adpcm.o adpcm_data.o
> OBJS-$(CONFIG_ADPCM_IMA_DK3_DECODER) += adpcm.o adpcm_data.o
> OBJS-$(CONFIG_ADPCM_IMA_DK4_DECODER) += adpcm.o adpcm_data.o
> diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c
> index f5d20ddd81..9ddcba6661 100644
> --- a/libavcodec/adpcm.c
> +++ b/libavcodec/adpcm.c
> @@ -14,6 +14,7 @@
> * THP ADPCM decoder by Marco Gerards (mgerards at xs4all.nl)
> * Argonaut Games ADPCM decoder by Zane van Iperen (zane at zanevaniperen.com)
> * Simon & Schuster Interactive ADPCM decoder by Zane van Iperen
> (zane at zanevaniperen.com)
> + * Ubisoft ADPCM decoder by Zane van Iperen (zane at zanevaniperen.com)
> *
> * This file is part of FFmpeg.
> *
> @@ -150,6 +151,14 @@ static av_cold int adpcm_decode_init(AVCodecContext *
> avctx)
> c->status[1].predictor = av_clip_intp2(AV_RL32(avctx->extradata
> + 4), 18);
> }
> break;
> + case AV_CODEC_ID_ADPCM_IMA_APM:
> + if (avctx->extradata && avctx->extradata_size >= 16) {
> + c->status[0].predictor = AV_RN32(avctx->extradata + 0);
> + c->status[0].step_index = AV_RN32(avctx->extradata + 4);
> + c->status[1].predictor = AV_RN32(avctx->extradata + 8);
> + c->status[1].step_index = AV_RN32(avctx->extradata + 12);
This must not be in native endianess. But in fixed one, eithere LE or BE.
> + }
> + break;
> case AV_CODEC_ID_ADPCM_IMA_WS:
> if (avctx->extradata && avctx->extradata_size >= 2)
> c->vqa_version = AV_RL16(avctx->extradata);
> @@ -168,6 +177,7 @@ static av_cold int adpcm_decode_init(AVCodecContext *
> avctx)
>
> switch(avctx->codec->id) {
> case AV_CODEC_ID_ADPCM_AICA:
> + case AV_CODEC_ID_ADPCM_IMA_APM:
> case AV_CODEC_ID_ADPCM_IMA_DAT4:
> case AV_CODEC_ID_ADPCM_IMA_QT:
> case AV_CODEC_ID_ADPCM_IMA_WAV:
> @@ -665,6 +675,7 @@ static int get_nb_samples(AVCodecContext *avctx,
> GetByteContext *gb,
> case AV_CODEC_ID_ADPCM_YAMAHA:
> case AV_CODEC_ID_ADPCM_AICA:
> case AV_CODEC_ID_ADPCM_IMA_SSI:
> + case AV_CODEC_ID_ADPCM_IMA_APM:
> nb_samples = buf_size * 2 / ch;
> break;
> }
> @@ -1227,6 +1238,18 @@ static int adpcm_decode_frame(AVCodecContext *avctx,
> void *data,
> *samples++ = adpcm_ima_qt_expand_nibble(&c->status[st], v &
> 0x0F, 3);
> }
> break;
> + case AV_CODEC_ID_ADPCM_IMA_APM:
> + for (channel = 0, n = 0; bytestream2_get_bytes_left(&gb) > 0;) {
> + int v = bytestream2_get_byteu(&gb);
> + *samples_p[channel]++ =
> adpcm_ima_qt_expand_nibble(&c->status[channel], v >> 4, 3);
> + *samples_p[channel]++ =
> adpcm_ima_qt_expand_nibble(&c->status[channel], v & 0x0F, 3);
> + channel = (channel + 1) % avctx->channels;
> + }
> +
> + /* Changing samples_p changes the frame itself. Undo any damage. */
> + for (channel = 0; channel < avctx->channels; channel++)
> + samples_p[channel] -= nb_samples;
> + break;
> case AV_CODEC_ID_ADPCM_IMA_OKI:
> while (bytestream2_get_bytes_left(&gb) > 0) {
> int v = bytestream2_get_byteu(&gb);
> @@ -1965,6 +1988,7 @@ ADPCM_DECODER(AV_CODEC_ID_ADPCM_EA_R3,
> sample_fmts_s16p, adpcm_ea_r3,
> ADPCM_DECODER(AV_CODEC_ID_ADPCM_EA_XAS, sample_fmts_s16p,
> adpcm_ea_xas, "ADPCM Electronic Arts XAS");
> ADPCM_DECODER(AV_CODEC_ID_ADPCM_IMA_AMV, sample_fmts_s16,
> adpcm_ima_amv, "ADPCM IMA AMV");
> ADPCM_DECODER(AV_CODEC_ID_ADPCM_IMA_APC, sample_fmts_s16,
> adpcm_ima_apc, "ADPCM IMA CRYO APC");
> +ADPCM_DECODER(AV_CODEC_ID_ADPCM_IMA_APM, sample_fmts_s16p,
> adpcm_ima_apm, "ADPCM IMA Ubisoft APM");
> ADPCM_DECODER(AV_CODEC_ID_ADPCM_IMA_DAT4, sample_fmts_s16,
> adpcm_ima_dat4, "ADPCM IMA Eurocom DAT4");
> ADPCM_DECODER(AV_CODEC_ID_ADPCM_IMA_DK3, sample_fmts_s16,
> adpcm_ima_dk3, "ADPCM IMA Duck DK3");
> ADPCM_DECODER(AV_CODEC_ID_ADPCM_IMA_DK4, sample_fmts_s16,
> adpcm_ima_dk4, "ADPCM IMA Duck DK4");
> diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c
> index 103f34fd32..6f543ea96f 100644
> --- a/libavcodec/allcodecs.c
> +++ b/libavcodec/allcodecs.c
> @@ -599,6 +599,7 @@ extern AVCodec ff_adpcm_g726le_encoder;
> extern AVCodec ff_adpcm_g726le_decoder;
> extern AVCodec ff_adpcm_ima_amv_decoder;
> extern AVCodec ff_adpcm_ima_apc_decoder;
> +extern AVCodec ff_adpcm_ima_apm_decoder;
> extern AVCodec ff_adpcm_ima_dat4_decoder;
> extern AVCodec ff_adpcm_ima_dk3_decoder;
> extern AVCodec ff_adpcm_ima_dk4_decoder;
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 368341ba93..f9d1af6ef9 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -549,6 +549,7 @@ enum AVCodecID {
> AV_CODEC_ID_ADPCM_ARGO,
> AV_CODEC_ID_ADPCM_IMA_SSI,
> AV_CODEC_ID_ADPCM_ZORK,
> + AV_CODEC_ID_ADPCM_IMA_APM,
>
> /* AMR */
> AV_CODEC_ID_AMR_NB = 0x12000,
> diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c
> index 609c7501fd..c5c379885f 100644
> --- a/libavcodec/codec_desc.c
> +++ b/libavcodec/codec_desc.c
> @@ -2318,6 +2318,13 @@ static const AVCodecDescriptor codec_descriptors[] =
> {
> .long_name = NULL_IF_CONFIG_SMALL("ADPCM Zork"),
> .props = AV_CODEC_PROP_INTRA_ONLY | AV_CODEC_PROP_LOSSY,
> },
> + {
> + .id = AV_CODEC_ID_ADPCM_IMA_APM,
> + .type = AVMEDIA_TYPE_AUDIO,
> + .name = "adpcm_ima_apm",
> + .long_name = NULL_IF_CONFIG_SMALL("ADPCM IMA Ubisoft APM"),
> + .props = AV_CODEC_PROP_INTRA_ONLY | AV_CODEC_PROP_LOSSY,
> + },
>
> /* AMR */
> {
> diff --git a/libavcodec/version.h b/libavcodec/version.h
> index 5c8147f681..b534b9260d 100644
> --- a/libavcodec/version.h
> +++ b/libavcodec/version.h
> @@ -28,7 +28,7 @@
> #include "libavutil/version.h"
>
> #define LIBAVCODEC_VERSION_MAJOR 58
> -#define LIBAVCODEC_VERSION_MINOR 70
> +#define LIBAVCODEC_VERSION_MINOR 71
> #define LIBAVCODEC_VERSION_MICRO 100
>
> #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \
> --
> 2.17.1
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
More information about the ffmpeg-devel
mailing list