Module: ngircd.git Branch: master Commit: 1c7b9dbe931ecbb0f355ab66c9b422f7e6001058 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=1c7b9...

Author: Florian Westphal fw@strlen.de Date: Sat Jan 10 00:44:34 2009 +0100

remove unneeded LOG_DEBUG when not compiling with DEBUG support

when ngircd is build without DEBUG enabled, LOG_DEBUG messages are always discarded.

To avoid the extra code, ngircd has a LogDebug() wrapper which gets removed by the compiler when compiling without DEBUG defined.

Update a few functings which were using the Log(LOG_DEBUG, .. interface directly without #ifdef DEBUG guards.

text data bss dec hex filename 127748 1900 28280 157928 268e8 ngircd.before 126836 1896 28280 157012 26554 ngircd.after

---

src/ngircd/channel.c | 2 +- src/ngircd/client.c | 2 +- src/ngircd/irc-login.c | 6 +++--- src/ngircd/irc-mode.c | 4 ++-- src/ngircd/ngircd.c | 2 +- src/ngircd/parse.c | 2 +- src/ngircd/resolve.c | 4 +--- 7 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/src/ngircd/channel.c b/src/ngircd/channel.c index 8caa81a..46e42ac 100644 --- a/src/ngircd/channel.c +++ b/src/ngircd/channel.c @@ -876,7 +876,7 @@ Add_Client( CHANNEL *Chan, CLIENT *Client ) cl2chan->next = My_Cl2Chan; My_Cl2Chan = cl2chan;

- Log( LOG_DEBUG, "User "%s" joined channel "%s".", Client_Mask( Client ), Chan->name ); + LogDebug("User "%s" joined channel "%s".", Client_Mask(Client), Chan->name);

return cl2chan; } /* Add_Client */ diff --git a/src/ngircd/client.c b/src/ngircd/client.c index 1d01f78..fbe130a 100644 --- a/src/ngircd/client.c +++ b/src/ngircd/client.c @@ -1047,7 +1047,7 @@ Generate_MyToken( CLIENT *Client ) else c = (CLIENT *)c->next; } Client->mytoken = token; - Log( LOG_DEBUG, "Assigned token %d to server "%s".", token, Client->id ); + LogDebug("Assigned token %d to server "%s".", token, Client->id); } /* Generate_MyToken */

diff --git a/src/ngircd/irc-login.c b/src/ngircd/irc-login.c index 943612e..1091852 100644 --- a/src/ngircd/irc-login.c +++ b/src/ngircd/irc-login.c @@ -242,7 +242,7 @@ IRC_NICK( CLIENT *Client, REQUEST *Req ) Client_Type(target) != CLIENT_SERVICE && Client_Type(target) != CLIENT_SERVER) { /* New client */ - Log( LOG_DEBUG, "Connection %d: got valid NICK command ...", + LogDebug("Connection %d: got valid NICK command ...", Client_Conn( Client ));

/* Register new nickname of this client */ @@ -723,7 +723,7 @@ IRC_PONG(CLIENT *Client, REQUEST *Req)

/* The connection timestamp has already been updated when the data has * been read from so socket, so we don't need to update it here. */ - +#ifdef DEBUG if (Client_Conn(Client) > NONE) Log(LOG_DEBUG, "Connection %d: received PONG. Lag: %ld seconds.", @@ -732,7 +732,7 @@ IRC_PONG(CLIENT *Client, REQUEST *Req) else Log(LOG_DEBUG, "Connection %d: received PONG.", Client_Conn(Client)); - +#endif return CONNECTED; } /* IRC_PONG */

diff --git a/src/ngircd/irc-mode.c b/src/ngircd/irc-mode.c index ed70a9b..aa979ac 100644 --- a/src/ngircd/irc-mode.c +++ b/src/ngircd/irc-mode.c @@ -531,7 +531,7 @@ Channel_Mode( CLIENT *Client, REQUEST *Req, CLIENT *Origin, CHANNEL *Channel ) strlcat(the_args, " ", sizeof(the_args)); strlcat(the_args, Client_ID(client), sizeof(the_args)); strlcat(the_modes, x, sizeof(the_modes)); - Log(LOG_DEBUG, "User "%s": Mode change on %s, now "%s"", + LogDebug("User "%s": Mode change on %s, now "%s"", Client_Mask(client), Channel_Name(Channel), Channel_UserModes(Channel, client)); } } else { @@ -539,7 +539,7 @@ Channel_Mode( CLIENT *Client, REQUEST *Req, CLIENT *Origin, CHANNEL *Channel ) retval = set ? Channel_ModeAdd(Channel, x[0]) : Channel_ModeDel(Channel, x[0]); if (retval) { strlcat(the_modes, x, sizeof(the_modes)); - Log(LOG_DEBUG, "Channel %s: Mode change, now "%s".", Channel_Name(Channel), Channel_Modes(Channel)); + LogDebug("Channel %s: Mode change, now "%s".", Channel_Name(Channel), Channel_Modes(Channel)); } }

diff --git a/src/ngircd/ngircd.c b/src/ngircd/ngircd.c index b564295..ad1168a 100644 --- a/src/ngircd/ngircd.c +++ b/src/ngircd/ngircd.c @@ -316,7 +316,7 @@ main( int argc, const char *argv[] ) #ifdef ZLIB strlcat( NGIRCd_ProtoID, "Z", sizeof NGIRCd_ProtoID ); #endif - Log( LOG_DEBUG, "Protocol and server ID is "%s".", NGIRCd_ProtoID ); + LogDebug("Protocol and server ID is "%s".", NGIRCd_ProtoID);

/* Vordefinierte Channels anlegen */ Channel_InitPredefined( ); diff --git a/src/ngircd/parse.c b/src/ngircd/parse.c index ec856a0..6801997 100644 --- a/src/ngircd/parse.c +++ b/src/ngircd/parse.c @@ -180,7 +180,7 @@ Parse_Request( CONN_ID Idx, char *Request ) ptr = strchr( Request, ' ' ); if( ! ptr ) { - Log( LOG_DEBUG, "Connection %d: Parse error: prefix without command!?", Idx ); + LogDebug("Connection %d: Parse error: prefix without command!?", Idx); return Conn_WriteStr( Idx, "ERROR :Prefix without command!?" ); } *ptr = '\0'; diff --git a/src/ngircd/resolve.c b/src/ngircd/resolve.c index 999ef99..e7f73ad 100644 --- a/src/ngircd/resolve.c +++ b/src/ngircd/resolve.c @@ -14,8 +14,6 @@

#include "portab.h"

-static char UNUSED id[] = "$Id: resolve.c,v 1.29 2008/02/26 22:04:17 fw Exp $"; - #include "imp.h" #include <assert.h> #include <errno.h> @@ -92,7 +90,7 @@ Resolve_Addr(RES_STAT * s, const ng_ipaddr_t *Addr, int identsock,

pid = Resolver_fork(pipefd); if (pid > 0) { - Log(LOG_DEBUG, "Resolver for %s created (PID %d).", ng_ipaddr_tostr(Addr), pid); + LogDebug("Resolver for %s created (PID %d).", ng_ipaddr_tostr(Addr), pid);

s->pid = pid; s->resolver_fd = pipefd[0];