[FFmpeg-devel] [PATCH] Optimization of original IFF codec

Sebastian Vater cdgs.basty
Fri Apr 23 12:26:37 CEST 2010


Arg, was too fast, sorry, please review this one instead.

Forgot to inline the plane decoder and the declare function prototype as
const unsigned instead of signed...

Sebastian Vater a ?crit :
> Hi again!
>
> Sebastian Vater a ?crit :
>   
>> I'll do a patch for this soon, but I want to mention though, that the
>> code in IFF does fiddle with bit-planar modes, thus the data is expected
>> to be calculated bit-wise. In that case I find it more readable to use
>> << and >> instead of * and /.
>>   
>>     
> So here is the patch, converting signed to unsigned where appreciated
> for optimization.
>
> I also tested this patch against breaking anything (at least ASH.LBM
> still displays correctly)...so please review it!
>   
-- 

Best regards,
                   :-) Basty/CDGS (-:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: iff-optimize.patch
Type: text/x-diff
Size: 4779 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100423/a9bb9909/attachment.patch>



More information about the ffmpeg-devel mailing list