[FFmpeg-devel] [PATCH] ADPCM IMA Radical decoder

[Paul B Mahol]

On 4/23/13, James Almer <jamrial at gmail.com> wrote:
> On 23/04/13 6:27 PM, James Almer wrote:
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/Makefile     |  1 +
>>  libavcodec/adpcm.c      | 31 +++++++++++++++++++++++++++++++
>>  libavcodec/allcodecs.c  |  1 +
>>  libavcodec/avcodec.h    |  1 +
>>  libavcodec/codec_desc.c |  7 +++++++
>>  libavcodec/utils.c      |  2 ++
>>  libavcodec/version.h    |  2 +-
>>  7 files changed, 44 insertions(+), 1 deletion(-)
>
> This is another codec supported by the RSD container.
> I can either resubmit that one decoder with support for this adpcm codec, or
> submit
> a separate patch that adds the needed changes.
>
> Also, to make things simpler since I've have submitted two separate branches
>
> (THP/RSD/Redspark/IMA_RAD and ADP/DTK, both with conflicting version bumps),
> i can
> put them in my public repo for merging after they get reviewed.
>
> Regards.


This one too LGTM.

You could merge all demuxer/codecs in single brach so that merging is
simplier and less work for
you and merger when rebasing and fixing conflicts (beacuse of
Changelog changes, etc)