[FFmpeg-cvslog] r22937 - in trunk: libavcodec/amrnbdec.c libavcodec/atrac1.c libavcodec/audioconvert.c libavcodec/qcelpdata.h libavcodec/qcelpdec.c libavcodec/ra288.c libavcodec/sipr.c libavcodec/sipr16k.c libavco...

[Ronald S. Bultje]

Hi,

On Sat, Apr 24, 2010 at 7:15 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Sat, Apr 24, 2010 at 04:43:19PM -0400, Ronald S. Bultje wrote:
>> Hi,
> [...]
>> ?common.h | ? ?2 +-
>> ?1 file changed, 1 insertion(+), 1 deletion(-)
>> 1c9e93cda8ce3e1e41630d14efc97d7528d75d41 ?clip.patch
>> Index: libavutil/common.h
>> ===================================================================
>> --- libavutil/common.h ? ? ? ?(revision 22937)
>> +++ libavutil/common.h ? ? ? ?(working copy)
>> @@ -151,7 +151,7 @@
>> ? */
>> ?static inline av_const int32_t av_clipl_int32(int64_t a)
>> ?{
>> - ? ?if ((a+2147483648) & ~2147483647) return (a>>63) ^ 2147483647;
>> + ? ?if ((a+2147483648U) & ~INT64_C(4294967295)) return (a>>63) ^ 2147483647;
>> ? ? ?else ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?return a;
>
> i think this would be more readable when written in hexadecimal

Fair enough, see attached.

Would you like me to make the same changes to clip8/16?

Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clip.patch
Type: application/octet-stream
Size: 457 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100425/e304bc5b/attachment-0001.obj>