On Tue, Apr 4, 2017 at 6:24 PM, Diego Biurrun <di...@biurrun.de> wrote:
> 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'.
ok so a normal %X should do
--
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
