[FFmpeg-soc] [soc]: r657 - matroska/matroskaenc.c

David Conrad umovimus at gmail.com
Fri Aug 10 04:14:43 CEST 2007


On Aug 9, 2007, at 9:30 PM, Aurelien Jacobs wrote:

> On Fri, 10 Aug 2007 01:37:13 +0200 (CEST)
> conrad <subversion at mplayerhq.hu> wrote:
>
>> Author: conrad
>> Date: Fri Aug 10 01:37:13 2007
>> New Revision: 657
>>
>> Log:
>> Add some debug logging
>>
>>
>> Modified:
>>    matroska/matroskaenc.c
>>
>> Modified: matroska/matroskaenc.c
>> ===================================================================== 
>> =========
>> --- matroska/matroskaenc.c	(original)
>> +++ matroska/matroskaenc.c	Fri Aug 10 01:37:13 2007
>> @@ -601,6 +601,8 @@ static void mkv_write_block(AVFormatCont
>>      MatroskaMuxContext *mkv = s->priv_data;
>>      ByteIOContext *pb = &s->pb;
>>
>> +    av_log(s, AV_LOG_DEBUG, "Writing block at offset %llu, size % 
>> d, pts %lld, dts %lld, duration %d, flags %d\n",
>> +           url_ftell(pb), pkt->size, pkt->pts, pkt->dts, pkt- 
>> >duration, flags);
>
> Using %llu and %lld is not appropriate to display (u)int64_t.
> It will at least generate a warning on 64 bits arch IIRC.
> Instead you should use %"PRId64" and %"PRIu64".
> (That apply to other usage of %ll* in this file).

Fixed.



More information about the FFmpeg-soc mailing list