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

Aurelien Jacobs aurel at gnuage.org
Fri Aug 10 03:30:53 CEST 2007


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).

Aurel



More information about the FFmpeg-soc mailing list