On Tue, Apr 04, 2017 at 05:47:35PM +0200, Vittorio Giovara wrote:
> On Tue, Apr 4, 2017 at 5:10 PM, Diego Biurrun <di...@biurrun.de> wrote:
> > On Tue, Apr 04, 2017 at 04:39:23PM +0200, Vittorio Giovara wrote:
> >> libavcodec/cfhd.c:258:16: warning: format specifies
> >> type 'unsigned short' but the argument has type 'int' [-Wformat]
> >> --- a/libavcodec/cfhd.c
> >> +++ b/libavcodec/cfhd.c
> >> @@ -254,11 +254,11 @@ static int parse_tag(CFHDContext *s, GetByteContext
> >> *gb,
> >>
> >> if (abs_tag8 >= 0x60 && abs_tag8 <= 0x6F) {
> >> - av_log(s->avctx, AV_LOG_DEBUG, "large len %"PRIX16"\n",
> >> + av_log(s->avctx, AV_LOG_DEBUG, "large len %"PRIX32"\n",
> >> ((tagu & 0xFF) << 16) | data);
> >
> > The warning does not match your change. tagu is uint16_t. I don't remember
> > what it gets promoted to offhand, but probably not uint32_t.
>
> it gets promoted to 'int'
So use the correct conversion specifier for 'int'.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
