[FFmpeg-devel] [PATCH 1/2] lavc/mpegvideo_enc: mark default_mv_penalty as const

Michael Niedermayer michael at niedermayer.cc
Tue Feb 9 00:35:33 EET 2021


On Mon, Feb 08, 2021 at 10:53:20AM +0100, Anton Khirnov wrote:
> Nothing is ever written into it.
> ---
>  libavcodec/mpegvideo_enc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
> index 34dcf8c313..a27c80ca37 100644
> --- a/libavcodec/mpegvideo_enc.c
> +++ b/libavcodec/mpegvideo_enc.c
> @@ -82,7 +82,7 @@ static int sse_mb(MpegEncContext *s);
>  static void denoise_dct_c(MpegEncContext *s, int16_t *block);
>  static int dct_quantize_trellis_c(MpegEncContext *s, int16_t *block, int n, int qscale, int *overflow);
>  
> -static uint8_t default_mv_penalty[MAX_FCODE + 1][MAX_DMV * 2 + 1];
> +static const uint8_t default_mv_penalty[MAX_FCODE + 1][MAX_DMV * 2 + 1];
>  static uint8_t default_fcode_tab[MAX_MV * 2 + 1];

if its optimal to have this the zero table instead of some non zero content
then i suggest to rename it to zero_mv_penalty

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210208/9d1d0bd1/attachment.sig>


More information about the ffmpeg-devel mailing list