[FFmpeg-devel] [PATCH] IFF: Merge lots of duplicate code into a single function

Sebastian Vater cdgs.basty
Sat May 15 03:48:02 CEST 2010


Peter Ross a ?crit :
> On Sat, May 15, 2010 at 03:03:12AM +0200, Sebastian Vater wrote:
>   
>> I noticed that the patch for the duplicate code alone is pretty long,
>> though.
>>
>> So I decided to do it as a separate patch.
>>
>> This patch creates a static function for byterun1 decoding and changes
>> decode_frame_byterun1 to use the new static function instead.
>>
>> This makes code a lot shorter.
>>     
>
> Looks good.
>   

I just found a small error forgotting to pass buf_end, so here's an
updated patch.

-- 

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: iff-function-merge.patch
Type: text/x-patch
Size: 5300 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100515/b0e71042/attachment.bin>



More information about the ffmpeg-devel mailing list