[FFmpeg-devel] [PATCH] In video_thread(), disable logging of rescaled timestamps if DEBUG is not enabled.

[Diego Biurrun]

On Thu, Jan 27, 2011 at 01:01:24AM +0100, Stefano Sabatini wrote:
> Avoid debug log spamming with -loglevel debug when the user is not
> debugging that code.
> 
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -58,6 +58,7 @@
>  
> +//#define DEBUG
>  //#define DEBUG_SYNC
>  
> @@ -1856,12 +1857,14 @@ static int video_thread(void *arg)
>  
>          if (av_cmp_q(tb, is->video_st->time_base)) {
> -            int64_t pts1 = pts_int;
> +            av_unused int64_t pts1 = pts_int;
>              pts_int = av_rescale_q(pts_int, tb, is->video_st->time_base);
> +#ifdef DEBUG
>              av_log(NULL, AV_LOG_DEBUG, "video_thread(): "
>                     "tb:%d/%d pts:%"PRId64" -> tb:%d/%d pts:%"PRId64"\n",
>                     tb.num, tb.den, pts1,
>                     is->video_st->time_base.num, is->video_st->time_base.den, pts_int);
> +#endif

dprintf

And if you move the function call into the av_log, you skip it in the
non-debug case and can avoid the av_unused I think.

Diego