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...@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
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
