On Fri, Nov 21, 2014 at 12:57 PM, Vittorio Giovara
<vittorio.giov...@gmail.com> wrote:
> From: Luca Barbato <lu_z...@gentoo.org>
>
> Might spare few cycles if the compiler is naive and
> makes the function more readable.
> ---
>  libavcodec/aacdec.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c
> index d2d51f5..6968102 100644
> --- a/libavcodec/aacdec.c
> +++ b/libavcodec/aacdec.c
> @@ -1419,13 +1419,12 @@ static void decode_mid_side_stereo(ChannelElement 
> *cpe, GetBitContext *gb,
>                                     int ms_present)
>  {
>      int idx;
> +    int max_idx = cpe->ch[0].ics.num_window_groups * cpe->ch[0].ics.max_sfb;
>      if (ms_present == 1) {
> -        for (idx = 0;
> -             idx < cpe->ch[0].ics.num_window_groups * cpe->ch[0].ics.max_sfb;
> -             idx++)
> +        for (idx = 0; idx < max_idx; idx++)
>              cpe->ms_mask[idx] = get_bits1(gb);
>      } else if (ms_present == 2) {
> -        memset(cpe->ms_mask, 1, cpe->ch[0].ics.num_window_groups * 
> cpe->ch[0].ics.max_sfb * sizeof(cpe->ms_mask[0]));
> +        memset(cpe->ms_mask, 1, max_idx * sizeof(cpe->ms_mask[0]));
>      }
>  }
>
> --
> 1.9.3 (Apple Git-50)
>

ping

-- 
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to