[FFmpeg-devel] [PATCH] Make h264idct.c compilation optional

Aurelien Jacobs aurel
Mon May 14 14:06:57 CEST 2007


On Mon, 14 May 2007 13:58:11 +0200
Panagiotis Issaris <takis.issaris at uhasselt.be> wrote:

> Hi,
> 
> The attached patch tries to make the h264idct.c compilation optional.
> Not meant for direct inclusion, as I am not sure about the correctness...
> 
> In dsputil_init() currently [1], for lowres decoding, when idct_algo's
> other than FF_IDCT_INT or FF_IDCT_AUTO are selected, h264 related
> functions (ff_h264_lowres_idct_put|add_c) are being used. Are these two
> functions in fact more generic then their name implies (in fact just
> misnamed), and should they be relocated to some other common file?
> 
> Or are they really H.264 specific functions? ... but in that case it
> seems weird that selecting f.e. FF_IDCT_VP3 or FF_IDCT_SIMPLEMMX would
> result in using these H.264 specific functions.

Patch seems ok, but I also would like to hear first what's the purpose
of ff_h264_lowres_idct_put/add_c (is it h264 only or more general ?).

Aurel




More information about the ffmpeg-devel mailing list