qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alan Conway <acon...@redhat.com>
Subject Re: nit in qpidd: diagnostic to stderr, not to stdout
Date Tue, 05 Dec 2006 20:46:35 GMT
Applied

On Tue, 2006-12-05 at 20:15 +0100, Jim Meyering wrote:
> This change makes qpidd write a diagnostic to stderr, not stdout.
> This demonstrates the problem:
> 
>   $ src/qpidd -9   2>/dev/null
>   Warning: skipping unrecognised option -9
>   WARNING: Persistence not enabled, no recovery of queues or messages.
>   Listening on port 5672...
> 
> IMHO, qpidd should fail in that case, too.
> 
> 2006-12-05  Jim Meyering  <meyering@redhat.com>
> 
> 	* lib/broker/Configuration.cpp (Configuration): Print diagnostic
> 	to stderr, not stdout.
> 
> Index: lib/broker/Configuration.cpp
> ===================================================================
> --- lib/broker/Configuration.cpp	(revision 482736)
> +++ lib/broker/Configuration.cpp	(working copy)
> @@ -52,7 +52,7 @@
>              matched = (*i)->parse(position, argv, argc);
>          }
>          if(!matched){
> -            std::cout<< "Warning: skipping unrecognised option " << argv[position]
<< std::endl;
> +            std::cerr<< "Warning: skipping unrecognised option " << argv[position]
<< std::endl;
>              position++;
>          }
>      }


Mime
View raw message