From: Florian Westphal Date: Sat, 5 Aug 2006 09:16:21 +0000 (+0000) Subject: if compiling without -DDEBUG, make LogDebug 'static inline' so gcc optimizes it away. X-Git-Tag: rel-0-11-0-pre2~125 X-Git-Url: https://arthur.barton.de/gitweb/?a=commitdiff_plain;h=83caef45986bb35b8d0d0741cf7b8a0278f2780e;p=ngircd.git if compiling without -DDEBUG, make LogDebug 'static inline' so gcc optimizes it away. --- diff --git a/src/ngircd/log.c b/src/ngircd/log.c index af8bcc60..9c4daf55 100644 --- a/src/ngircd/log.c +++ b/src/ngircd/log.c @@ -14,7 +14,7 @@ #include "portab.h" -static char UNUSED id[] = "$Id: log.c,v 1.61 2006/07/23 23:23:45 alex Exp $"; +static char UNUSED id[] = "$Id: log.c,v 1.62 2006/08/05 09:16:21 fw Exp $"; #include "imp.h" #include @@ -166,25 +166,21 @@ Log_Exit( void ) * Log function for debug messages. * This function is only functional when the program is compiled with debug * code enabled; otherwise it is an empty function which the compiler will - * hopefully mangle down to "nothing". Therefore you should use LogDebug(...) - * in favor to Log(LOG_DEBUG, ...). + * hopefully mangle down to "nothing" (see log.h). Therefore you should use + * LogDebug(...) in favor to Log(LOG_DEBUG, ...). * @param Format Format string like printf(). * @param ... Further arguments. */ +#ifdef DEBUG # ifdef PROTOTYPES GLOBAL void -#ifdef DEBUG LogDebug( const char *Format, ... ) -#else -LogDebug( UNUSED const char *Format, ... ) -#endif /* DEBUG */ # else GLOBAL void LogDebug( Format, va_alist ) const char *Format; va_dcl # endif /* PROTOTYPES */ -#ifdef DEBUG { char msg[MAX_LOG_MSG_LEN]; va_list ap; @@ -199,14 +195,9 @@ va_dcl va_end( ap ); Log(LOG_DEBUG, "%s", msg); } -#else -{ - /* Do nothing. - * The compiler should optimize this out, please ;-) */ -} #endif /* DEBUG */ - + /** * Logging function of ngIRCd. * This function logs messages to the console and/or syslog, whichever is diff --git a/src/ngircd/log.h b/src/ngircd/log.h index 00a8a0f7..529b1640 100644 --- a/src/ngircd/log.h +++ b/src/ngircd/log.h @@ -8,7 +8,7 @@ * (at your option) any later version. * Please read the file COPYING, README and AUTHORS for more information. * - * $Id: log.h,v 1.19 2006/02/08 17:33:28 fw Exp $ + * $Id: log.h,v 1.20 2006/08/05 09:16:21 fw Exp $ * * Logging functions (header) */ @@ -39,7 +39,13 @@ GLOBAL void Log_Init PARAMS(( bool Daemon_Mode )); GLOBAL void Log_Exit PARAMS(( void )); GLOBAL void Log PARAMS(( int Level, const char *Format, ... )); + +#ifdef DEBUG GLOBAL void LogDebug PARAMS(( const char *Format, ... )); +#else +static inline void LogDebug PARAMS(( UNUSED const char *Format, ... )){/* Do nothing. The compiler should optimize this out, please ;-) */} +#endif + GLOBAL void Log_Init_Resolver PARAMS(( void )); GLOBAL void Log_Exit_Resolver PARAMS(( void ));