[FFmpeg-devel] [PATCH] fix clicks in ADPCM IMA AMV decoder

Vladimir Voroshilov voroshil
Wed Oct 3 17:55:28 CEST 2007


Hi, All

Attached simle patches fixes clicks in audio when playing AMV files.

If i'm undestanding things correctly, predictor is first PCM sample to
start decoding from and thus it should be signed short value.

First patch with "both" suffix fixes assignment for both AMV and SMJPEG, while
second (marked as "case") adds special case for handling only AMV.

If preferred patch is ok please apply it.

P.S. Patches are simplified versions (with removed duplicated code)
version of patches used in amv-codec-tools project. Original code was
written by Dobrica Pavlinusic.


-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719
-------------- next part --------------
A non-text attachment was scrubbed...
Name: adpcm_clicking_fix_both.diff
Type: text/x-diff
Size: 507 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071003/760222bb/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: adpcm_clicking_fix_case.diff
Type: text/x-diff
Size: 628 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071003/760222bb/attachment-0001.diff>



More information about the ffmpeg-devel mailing list