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

[Kyle Swanson]

Hi,

On Sun, Mar 26, 2017 at 1:20 PM, Nicolas George <george at nsup.org> wrote:
>
> 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.


Thanks, new patch is attached. %" PRIu64 " should be correct.

>
>
> >                 avctx->bit_rate/1000, clipped_rate/1000);
> >          avctx->bit_rate = clipped_rate;
> >      }
>
> Regards,
>
> --
>   Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-libavcodec-opusenc-use-correct-format-specifiers.patch
Type: application/octet-stream
Size: 1603 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170326/c18c5bd1/attachment.obj>