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

Luca Barbato lu_zero
Sat Jan 29 11:53:52 CET 2011


On 01/29/2011 11:46 AM, Diego Biurrun wrote:
> IIRC (but I may well be wrong) the ff* tools do use internal symbols in
> other places.  Another option is to make the symbol or some variant of
> it public.

They should not. We have already not many examples and we shouldn't
suggest wrong (and soon-broken) behaviours.

>> Also it conflicts with the dprintf defined in stdio.h.
> 
> This is of course bad, but such a name clash should be addressed in
> FFmpeg anyway.  We can just rename it to dbg_printf, ff_dprintf or
> similar.

Agreed.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list