[FFmpeg-devel] [PATCH] avcodec/opus: Add {} over multiline if() body

Rostislav Pehlivanov atomnuker at gmail.com
Wed Jan 3 03:34:59 EET 2018


On 2 January 2018 at 22:34, Michael Niedermayer <michael at niedermayer.cc>
wrote:

> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/opus.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/opus.c b/libavcodec/opus.c
> index 9cbf4aed92..d00a17a7f1 100644
> --- a/libavcodec/opus.c
> +++ b/libavcodec/opus.c
> @@ -566,12 +566,12 @@ void ff_celt_bitalloc(CeltFrame *f, OpusRangeCoder
> *rc, int encode)
>      int bits2[CELT_MAX_BANDS];
>
>      /* Spread */
> -    if (opus_rc_tell(rc) + 4 <= f->framebits)
> +    if (opus_rc_tell(rc) + 4 <= f->framebits) {
>          if (encode)
>              ff_opus_rc_enc_cdf(rc, f->spread, ff_celt_model_spread);
>          else
>              f->spread = ff_opus_rc_dec_cdf(rc, ff_celt_model_spread);
> -    else
> +    } else
>          f->spread = CELT_SPREAD_NORMAL;
>
>      /* Initialize static allocation caps */
> --
> 2.15.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Add brackets after the } else and it'll look good to me.


More information about the ffmpeg-devel mailing list