[FFmpeg-cvslog] r22974 - trunk/libavcodec/iff.c

Ronald S. Bultje rsbultje
Tue May 4 00:08:48 CEST 2010


Hi,

On Tue, Apr 27, 2010 at 12:12 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Apr 27, 2010 at 09:52:20AM -0400, Ronald S. Bultje wrote:
>> On Mon, Apr 26, 2010 at 8:28 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> > On Tue, Apr 27, 2010 at 12:37:13AM +0200, rbultje wrote:
>> >> Author: rbultje
>> >> Date: Tue Apr 27 00:37:13 2010
>> >> New Revision: 22974
>> >>
>> >> Log:
>> >> Switch some ints to unsigned (they can only have positive values, this allows
>> >> compiler to optimize some math from mul/div to shr/shl).
>>
>> >> Also add a cast to
>> >> uint32_t when calling decodeplane32().
>> >
>> > and why?
>>
>> Silences a compiler warning.
>
> ->log

Added (a few days ago).

>> [..]
>> > never used in >> or /
>>
>> Hmk, would you like me to revert these parts? I confess that I didn't
>> check each individual one.
>
> yes i would like you to revert all that you dont check and all that make
> no difference

OK, all reverted.

Ronald



More information about the ffmpeg-cvslog mailing list