[FFmpeg-devel] [PATCH 07/11] Make denoise_dct_c and dct_quantize_trellis_c static.

Diego Biurrun diego
Tue Jan 25 01:47:00 CET 2011


On Tue, Jan 25, 2011 at 12:36:15AM +0100, Diego Elio Petten? wrote:
> 
> --- a/libavcodec/mpegvideo_common.h
> +++ b/libavcodec/mpegvideo_common.h
> @@ -40,8 +40,6 @@
>  
>  int dct_quantize_c(MpegEncContext *s, DCTELEM *block, int n, int qscale, int *overflow);
> -int dct_quantize_trellis_c(MpegEncContext *s, DCTELEM *block, int n, int qscale, int *overflow);
> -void  denoise_dct_c(MpegEncContext *s, DCTELEM *block);
>  
> --- a/libavcodec/mpegvideo_enc.c
> +++ b/libavcodec/mpegvideo_enc.c
> @@ -48,6 +48,8 @@
>  static int encode_picture(MpegEncContext *s, int picture_number);
>  static int dct_quantize_refine(MpegEncContext *s, DCTELEM *block, int16_t *weight, DCTELEM *orig, int n, int qscale);
>  static int sse_mb(MpegEncContext *s);
> +static void denoise_dct_c(MpegEncContext *s, DCTELEM *block);
> +static int dct_quantize_trellis_c(MpegEncContext *s, DCTELEM *block, int n, int qscale, int *overflow);
>  
>  /* enable all paranoid tests for rounding, overflows, etc... */
>  //#define PARANOID

I don't see you making the functions themselves static, which seems wrong.

Diego



More information about the ffmpeg-devel mailing list