[MPlayer-dev-eng] [PATCH] verbosity reduction in streaming, input, CPU detection

[Roberto Togni]

On Sun, 24 Sep 2006 18:22:39 +0200
Diego Biurrun <diego at biurrun.de> wrote:

> Hi,
> 
> these are a couple of verbosity reduction patches I have accumulated
> in my local tree.  I'd like to commit by the end of the week, comments
> welcome.
> 
> Diego

> Index: stream/http.c
> ===================================================================
> --- stream/http.c	(revision 19966)
> +++ stream/http.c	(working copy)
> @@ -891,7 +891,7 @@
>  	stream->streaming_ctrl->url = check4proxies(url);
>  	url_free(url);
>  	
> -	mp_msg(MSGT_OPEN, MSGL_INFO, "STREAM_HTTP(1), URL: %s\n",
> stream->url);
> +	mp_msg(MSGT_OPEN, MSGL_V, "STREAM_HTTP(1), URL: %s\n",
> stream->url); seekable = http_streaming_start(stream, file_format);
>  	if((seekable < 0) || (*file_format == DEMUXER_TYPE_ASF)) {
>  		streaming_ctrl_free(stream->streaming_ctrl);
> @@ -915,7 +915,7 @@
>  	stream->streaming_ctrl->url = check4proxies(url);
>  	url_free(url);
>  	
> -	mp_msg(MSGT_OPEN, MSGL_INFO, "STREAM_HTTP(2), URL: %s\n",
> stream->url);
> +	mp_msg(MSGT_OPEN, MSGL_V, "STREAM_HTTP(2), URL: %s\n",
> stream->url); seekable = http_streaming_start(stream, file_format);
>  	if(seekable < 0) {
>  		streaming_ctrl_free(stream->streaming_ctrl);

Ok

> Index: stream/network.c
> ===================================================================
> --- stream/network.c	(revision 19966)
> +++ stream/network.c	(working copy)
> @@ -440,7 +440,7 @@
>        stream_cache_size =
> (stream->streaming_ctrl->prebuffer_size/1024)*5;
> if( stream_cache_size<64 ) stream_cache_size = 64;	// 16KBytes
> min buffer }
> -    mp_msg(MSGT_NETWORK,MSGL_INFO,MSGTR_MPDEMUX_NW_CacheSizeSetTo,
> stream_cache_size);
> +    mp_msg(MSGT_NETWORK,MSGL_V,"Cache size set to %d kBytes.\n",
> stream_cache_size); }

This should stay

>  }
>  
> Index: stream/stream_rtsp.c
> ===================================================================
> --- stream/stream_rtsp.c	(revision 19966)
> +++ stream/stream_rtsp.c	(working copy)
> @@ -140,7 +140,7 @@
>    URL_t *url;
>    extern int index_mode;
>    
> -  mp_msg (MSGT_OPEN, MSGL_INFO, "STREAM_RTSP, URL: %s\n",
> stream->url);
> +  mp_msg (MSGT_OPEN, MSGL_V, "STREAM_RTSP, URL: %s\n", stream->url);

Ok

>    stream->streaming_ctrl = streaming_ctrl_new ();
>    if (!stream->streaming_ctrl)
>      return STREAM_ERROR;
> Index: stream/tcp.c
> ===================================================================
> --- stream/tcp.c	(revision 19966)
> +++ stream/tcp.c	(working copy)
> @@ -109,7 +109,7 @@
>  	if ( inet_addr(host)==INADDR_NONE )
>  #endif
>  	{
> -		if(verb)
> mp_msg(MSGT_NETWORK,MSGL_STATUS,MSGTR_MPDEMUX_NW_ResolvingHostForAF,
> host, af2String(af));
> +		if(verb) mp_msg(MSGT_NETWORK,MSGL_V,"Resolving %s
> for %s...\n", host, af2String(af)); 
>  #ifdef HAVE_GETHOSTBYNAME2
>  		hp=(struct hostent*)gethostbyname2( host, af );
> @@ -153,7 +153,7 @@
>  #else
>  	inet_ntop(af, our_s_addr, buf, 255);
>  #endif
> -	if(verb)
> mp_msg(MSGT_NETWORK,MSGL_STATUS,MSGTR_MPDEMUX_NW_ConnectingToServer,
> host, buf , port );
> +	if(verb) mp_msg(MSGT_NETWORK,MSGL_V,"Connecting to server
> %s[%s]: %d...\n", host, buf , port ); 

I think also this should stay there

[...]

> Index: cpudetect.c
> ===================================================================
> --- cpudetect.c	(revision 19966)
> +++ cpudetect.c	(working copy)

Why?

[...]

Ciao,
 Roberto