[FFmpeg-devel] [PATCH] Build extradata in adpcm_ms encoder

Daniel Verkamp daniel
Thu Oct 22 22:45:14 CEST 2009


On Thu, Oct 22, 2009 at 3:29 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Oct 22, 2009 at 10:58:58AM -0500, Daniel Verkamp wrote:
>> On Thu, Oct 22, 2009 at 6:01 AM, Diego Biurrun <diego at biurrun.de> wrote:
>> > On Wed, Oct 21, 2009 at 03:50:28PM -0500, Daniel Verkamp wrote:
>> >>
>> >> The addition of comments for AdaptCoeff1/2 is not really a related
>> >> change, but it took me several minutes to figure out why these
>> >> coefficients did not match the expected values, so I think it's
>> >> useful. ?The change that introduced this was r14173.
>> >
>> > Then it should of course be a separate patch. ?If it would have
>> > helped you figure this out, I think you can commit it.
>>
>> AdaptCoeff comments applied separately; rebased patch attached.
>
>> From d9b80812b866d3c1c64079911ea788cdbc3254af Mon Sep 17 00:00:00 2001
>> From: Daniel Verkamp <daniel at drv.nu>
>> Date: Wed, 21 Oct 2009 16:44:07 -0400
>> Subject: [PATCH] Build extradata in adpcm_ms encoder
>>
>> Fixes issue #1244
>> ---
>> ?libavcodec/adpcm.c ? ? ? ? ? ?| ? 12 +++++++++
>> ?tests/rotozoom.regression.ref | ? ?4 +-
>> ?tests/seek.regression.ref ? ? | ? 54 ++++++++++++++++++++--------------------
>> ?tests/vsynth.regression.ref ? | ? ?4 +-
>> ?4 files changed, 43 insertions(+), 31 deletions(-)
>
> ok

Applied.



More information about the ffmpeg-devel mailing list