[FFmpeg-devel] [PATCH] Common code for Indeo interactive

Aurelien Jacobs aurel
Fri Mar 20 02:04:51 CET 2009


On Thu, Mar 19, 2009 at 11:12:40PM +0100, Maxim wrote:
> Hello guys,
> 
> here is the 1st part of my patch making FFmpeg decode Indeo interactive
> (indeo4/indeo5) streams. Both codecs are very similar but use abit
> different bitstream formats, so I decided myself to put all common
> functions and tables together and submit them as a separate patch.
> The 2nd part will be the long-awaited indeo5 decoder...
> Please review!
> 
> [...]
> 
> /**
>  *  motion compensation without adding delta
>  *
>  *  @param  buf     [in,out] pointer to the block in the current frame receiving the result
>  *  @param  ref_buf [in] pointer to the corresponding block in the reference frame
>  *  @param  pitch   [in] pitch for moving to the next y line
>  *  @param  mc_type [in] interpolation type
>  */
> static void ivi_mc_8x8_no_delta (int16_t *buf, int16_t *ref_buf, uint32_t pitch, int mc_type)


This function probably is exactly the same as put_pixels_tab[1][mc_type]()
from dsputils. And you can probably also use dsputils for all other MC
functions.

Also your code would benefit from some vertical alignment all over the place,
in the code and the data tables.

Aurel




More information about the ffmpeg-devel mailing list