[FFmpeg-devel] [PATCH 07/10] aacsbr_fixed: Make fixed_{exp, log}_table static const
Ganesh Ajjanagadde
gajjanag at mit.edu
Sat Aug 22 04:03:06 CEST 2015
On Fri, Aug 21, 2015 at 9:21 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
> ---
> libavcodec/aacsbr_fixed.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/aacsbr_fixed.c b/libavcodec/aacsbr_fixed.c
> index 1fc7385..e048069 100644
> --- a/libavcodec/aacsbr_fixed.c
> +++ b/libavcodec/aacsbr_fixed.c
> @@ -78,7 +78,7 @@ static const int CONST_LN2 = Q31(0.6931471806/256); // ln(2)/256
> static const int CONST_RECIP_LN2 = Q31(0.7213475204); // 0.5/ln(2)
> static const int CONST_076923 = Q31(0.76923076923076923077f);
>
> -int fixed_log_table[10] =
> +static const int fixed_log_table[10] =
> {
> Q31(1.0/2), Q31(1.0/3), Q31(1.0/4), Q31(1.0/5), Q31(1.0/6),
> Q31(1.0/7), Q31(1.0/8), Q31(1.0/9), Q31(1.0/10), Q31(1.0/11)
> @@ -103,7 +103,7 @@ static int fixed_log(int x)
> return ret;
> }
>
> -int fixed_exp_table[7] =
> +static const int fixed_exp_table[7] =
> {
> Q31(1.0/2), Q31(1.0/6), Q31(1.0/24), Q31(1.0/120),
> Q31(1.0/720), Q31(1.0/5040), Q31(1.0/40320)
> --
> 1.9.1
Relatively recent code essentially from
f85bc147fb87de048ccc5767e186ac59ec0284ef.
Been over a month, so LGTM.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
More information about the ffmpeg-devel
mailing list