[MPlayer-dev-eng] [PATCH] libmplayer

Bryan Wilkerson bryanwilkerson at yahoo.com
Fri Jun 17 23:11:55 CEST 2005


--- The Wanderer <inverseparadox at comcast.net> wrote:

> Bryan Wilkerson 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), both
> because it precludes
> the possibility of sending output to stdout and -
> notably in the current
> case - because an underlying library should AFAIK
> never use stdout. Use
> mp_msg instead. 

Sorry, missed that one.  Attached is the updated
patch.

-bryan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-libmplayer.diff
Type: text/x-patch
Size: 81121 bytes
Desc: 2374417340-mplayer-libmplayer.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050617/a8f78779/attachment.bin>


More information about the MPlayer-dev-eng mailing list