[MPlayer-dev-eng] [PATCH] libmplayer
The Wanderer
inverseparadox at comcast.net
Fri Jun 17 23:22:01 CEST 2005
Nico Sabbi wrote:
> The Wanderer wrote:
>
>>> + if(!quiet)
>>> + {
>>> + // TODO : should probably have a Resumed msg
>>> + printf(" \n"); //clear pause and go to next line
>>
>> Please do not use printf except in debugging code, i.e. commented
>> out or inside #if 0 (and ideally not then either)
>
> IMO even if surrounded by #if 0/#endif blocks there's no reason not
> to replace it with mp_msg with MSGL_INFO loglevel
That would be my position as well (except perhaps for the MSGL), and I
might extend that even to commented-out lines (against, say, the
possibility of wanting to both send video/audio to stdout *and* get
debug output), but I believe that some of the people who do most of the
work here have disagreed with me.
--
The Wanderer
Warning: Simply because I argue an issue does not mean I agree with any
side of it.
A government exists to serve its citizens, not to control them.
More information about the MPlayer-dev-eng
mailing list