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

[Michael Niedermayer]

Hi

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;
START_TIMER
for(i=-40000; i<40000; i++)
    v+= av_clip_int16(i)
STOP_TIMER("foober")
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>