[FFmpeg-devel] [PATCH 07/11] Make denoise_dct_c and dct_quantize_trellis_c static.
Diego Biurrun
diego
Tue Jan 25 02:01:56 CET 2011
On Tue, Jan 25, 2011 at 01:57:10AM +0100, Diego Elio Petten? wrote:
> Il giorno mar, 25/01/2011 alle 01.47 +0100, Diego Biurrun ha scritto:
> > I don't see you making the functions themselves static, which seems
> > wrong.
>
> The static declaration should be enough for the compiler, and it is
> here:
>
> % nm libavcodec/mpegvideo_enc.o| grep denoise_
> 0000000000000a80 t denoise_dct_c
>
> But if you wish to make it more explicit, I'll send a patch.
I would prefer that, I find the static forward declaration and the
non-static definition confusing.
Diego
More information about the ffmpeg-devel
mailing list