[FFmpeg-devel] [PATCH] adpcm-ima-wav header and codec

[Timofei V. Bondarenko]

Michael Niedermayer wrote:
> Hi
> 
> On Wed, Oct 17, 2007 at 05:24:02PM +0400, Timofei V. Bondarenko wrote:
> [...]
>>> [...]
>>>> --- ffmpeg-svn/libavcodec/adpcm.c	2007-10-12 12:21:16.000000000 +0400
>>>> +++ ffmpeg/libavcodec/adpcm.c	2007-10-12 19:01:53.000000000 +0400
>>>> @@ -483,17 +483,17 @@
>>> looks ok
>> BTW these changes break regression tests. It is intentional. Should i do 
>> something about?
> 
> yes update the regresson tests checksums in the same patch which changes them
> 

Done. This patch fixes following issues:

>> In adpcm_encode_frame() predictor for right channel gets initialized by 2nd 
>> sample of the left channel instead of 1st sample of the right channel.
>>
>> The adpcm_decode_frame() loses first sample in block.
>> This sample initializes predictor and then gets lost instead of being 
>> copied to output.

Regards,
	Timofei.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: adpcm-ima-codec.patch
Type: text/x-patch
Size: 1979 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071102/4f003be3/attachment.bin>