[FFmpeg-devel] [PATCH] libavcodec/opusenc: use correct format specifiers

Nicolas George george at nsup.org
Sun Mar 26 21:20:33 EEST 2017


Thanks for the patch.

Le sextidi 6 germinal, an CCXXV, Kyle Swanson a écrit :
> Squelches the following compiler warnings:
> 
> libavcodec/opusenc.c:1051:16: warning: format specifies type 'long' but
> the argument has type 'long long' [-Wformat]
>                avctx->bit_rate/1000, clipped_rate/1000);
>                ^~~~~~~~~~~~~~~~~~~~
> libavcodec/opusenc.c:1051:38: warning: format specifies type 'long' but
> the argument has type 'long long' [-Wformat]
>                avctx->bit_rate/1000, clipped_rate/1000);
>                                      ^~~~~~~~~~~~~~~~~
> 
> Signed-off-by: Kyle Swanson <k at ylo.ph>
> ---
>  libavcodec/opusenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/opusenc.c b/libavcodec/opusenc.c
> index 56368db..292d060 100644
> --- a/libavcodec/opusenc.c
> +++ b/libavcodec/opusenc.c
> @@ -1047,7 +1047,7 @@ static av_cold int opus_encode_init(AVCodecContext *avctx)
>          avctx->bit_rate = coupled*(96000) + (s->channels - coupled*2)*(48000);
>      } else if (avctx->bit_rate < 6000 || avctx->bit_rate > 255000 * s->channels) {
>          int64_t clipped_rate = av_clip(avctx->bit_rate, 6000, 255000 * s->channels);

> -        av_log(avctx, AV_LOG_ERROR, "Unsupported bitrate %li kbps, clipping to %li kbps\n",
> +        av_log(avctx, AV_LOG_ERROR, "Unsupported bitrate %lli kbps, clipping to %lli kbps\n",

The old specifier was wrong, but %ll is wrong too: avctx->bit_rate is
int64_t, not long nor long long.

>                 avctx->bit_rate/1000, clipped_rate/1000);
>          avctx->bit_rate = clipped_rate;
>      }

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170326/e12d661c/attachment.sig>


More information about the ffmpeg-devel mailing list