[FFmpeg-devel] [PATCH] move fast dct and trellis quantize function pointers initialization
Wed Jun 20 10:12:14 CEST 2007
Michael Niedermayer wrote:
> On Wed, Jun 20, 2007 at 08:38:06AM +0100, M?ns Rullg?rd wrote:
>> Benoit Fouet <benoit.fouet at purplelabs.com> writes:
>>> the patch moves the encoder specific part of DCT_common_init under the
>>> same ifdef.
>>> it also removes the comment telling to do so :)
>> OK with me.
> approving pacthes for code maintained by others? didnt you just complain
> about me approving one for configure?
> anyway i dont mind it if you do, actually iam happy if people help
> reviewing and approving stuff but here the patch is not ok at all
> s->fast_dct_quantize= s->dct_quantize;
> is currently done after dct_quantize is set up (possibly to a mmx optimized
> variant) but after the patch its just set to the c variant ...
i can move the CONFIG_ENCODERS part after the arch dependant
would that be ok ?
Purple Labs S.A.
More information about the ffmpeg-devel