[FFmpeg-devel] [PATCH] fix clicks in ADPCM IMA AMV decoder
Vitor Sessak
vitor1001
Wed Oct 3 20:59:30 CEST 2007
Vladimir Voroshilov wrote:
> 2007/10/4, Vitor Sessak <vitor1001 at gmail.com>:
>> Hi
>>
>> Vladimir Voroshilov wrote:
>>>>>>>> Vladimir Voroshilov wrote:
>>>>>>>>> Hi, All
>>>>>>>>>
>>>>> Index: libavcodec/adpcm.c
>>>>> ===================================================================
>>>>> --- libavcodec/adpcm.c (revision 10652)
>>>>> +++ libavcodec/adpcm.c (working copy)
>>>>> @@ -1184,10 +1184,8 @@
>>>>> break;
>>>>> case CODEC_ID_ADPCM_IMA_AMV:
>>>>> case CODEC_ID_ADPCM_IMA_SMJPEG:
>>>>> - c->status[0].predictor = *src;
>>>>> - src += 2;
>>>>> - c->status[0].step_index = *src++;
>>>>> - src++; /* skip another byte before getting to the meat */
>>>>> + c->status[0].predictor = (signed short)bytestream_get_le16(&src);
>>>>> + c->status[0].step_index = bytestream_get_le16(&src);
>>>> I suppose the signed short cast is useless. If so, please remove it.
>>>> If you remove it, you could also align the = sign.
>>> Signed cast is required (exactly this signed cast removes clicks in sound).
>> Would just this do it?
>>
>> c->status[0].predictor = (signed) bytestream_get_le16(&src);
>
> No, "(signed)" cast does not work.
Ok... I promise I'll test it before suggesting next time :-)
-Vitor
More information about the ffmpeg-devel
mailing list