On Tue, Apr 04, 2017 at 05:47:35PM +0200, Vittorio Giovara wrote: > On Tue, Apr 4, 2017 at 5:10 PM, Diego Biurrun <[email protected]> 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 [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
