[FFmpeg-devel] [PATCH] Replace CLAMP_TO_SHORT with av_clip in adpcm.c

Michael Niedermayer michaelni
Sat Aug 11 12:02:38 CEST 2007


On Sat, Aug 11, 2007 at 02:13:51AM -0300, Ramiro Ribeiro Polla wrote:
> Aurelien Jacobs wrote:
> > Hi,
> >
> > $subj
> > This fixes issue84.
> >
> >   
> What about making av_clip_int16 in libavutil/common.h?
> There are many more places throughout FFmpeg where this is used.

what is the speed/filesize effect of such a change?

speed being:
volatile int v=0;
for(i=-40000; i<40000; i++)
    v+= av_clip_int16(i)
printf("%d", v);

filesize being object size with --enable-small


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070811/791abbd7/attachment.pgp>

More information about the ffmpeg-devel mailing list