[FFmpeg-devel] [PATCH] Yet more ALAC cleanup (#define removal)

[Michael Niedermayer]

Hi

On Wed, Aug 15, 2007 at 08:50:05PM +0200, Vitor Sessak wrote:
> Hi
>
> Reimar D?ffinger wrote:
>> Hello,
>> On Wed, Aug 15, 2007 at 08:22:59PM +0200, Vitor Sessak wrote:
>>> -#define SIGN_ONLY(v) \
>>> -                     ((v < 0) ? (-1) : \
>>> -                                ((v > 0) ? (1) : \
>>> -                                           (0)))
>>> +static inline int sign_only(int v)
>>> +{
>>> +    if (v < 0)
>>> +        return -1;
>>> +    else if (v > 0)
>>> +        return 1;
>>> +    else
>>> +        return 0;
>>> +}
>> Since you change it anyway, I'd suggest
>> if (v == 0) return 0;
>> return FFSIGN(v);
>> Though benchmarking it might be a good idea unless it is really
>> uncritical.
>> Greetings,
>> Reimar D?ffinger
>
> Nice suggestion, but I don't think it is worth benchmarking. Unless the 
> compiler is very stupid, 

anyone did a IQ test with gcc lately?

anyway, if you benchmark it, also try
(v>>31)-((-v)>>31)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- 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/20070815/bd44fba9/attachment.pgp>