On 04/04/2017 17:47, 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 a normal %d is enough
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to