- ngIRCd GIT Updates - lists.barton.de

Alexander Barton : Only #include resolve.h if it is really needed
by alex＠arthur.barton.de 14 Jul '10

14 Jul '10

Module: ngircd.git Branch: master Commit: cc336b75580d012c1caee58a252d1ad14b2cee23 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=cc336b75… Author: Alexander Barton <alex(a)barton.de> Date: Tue May 18 17:06:31 2010 +0200 Only #include resolve.h if it is really needed --- src/ngircd/channel.c | 1 - src/ngircd/client.c | 1 - src/ngircd/conf.c | 1 - src/ngircd/irc-channel.c | 1 - src/ngircd/irc-info.c | 1 - src/ngircd/irc-login.c | 1 - src/ngircd/irc-mode.c | 1 - src/ngircd/irc-oper.c | 1 - src/ngircd/irc-server.c | 1 - src/ngircd/irc.c | 1 - src/ngircd/ngircd.c | 1 - src/ngircd/numeric.c | 1 - 12 files changed, 0 insertions(+), 12 deletions(-) diff --git a/src/ngircd/channel.c b/src/ngircd/channel.c index 6e9e0f4..0fa38de 100644 --- a/src/ngircd/channel.c +++ b/src/ngircd/channel.c @@ -34,7 +34,6 @@ #include "imp.h" #include "irc-write.h" -#include "resolve.h" #include "conf.h" #include "hash.h" #include "lists.h" diff --git a/src/ngircd/client.c b/src/ngircd/client.c index 7cf10da..e6dbf15 100644 --- a/src/ngircd/client.c +++ b/src/ngircd/client.c @@ -35,7 +35,6 @@ #include <imp.h> #include "ngircd.h" #include "channel.h" -#include "resolve.h" #include "conf.h" #include "hash.h" #include "irc-write.h" diff --git a/src/ngircd/conf.c b/src/ngircd/conf.c index 694b5d4..f9db5c7 100644 --- a/src/ngircd/conf.c +++ b/src/ngircd/conf.c @@ -44,7 +44,6 @@ #include "defines.h" #include "log.h" #include "match.h" -#include "resolve.h" #include "tool.h" #include "exp.h" diff --git a/src/ngircd/irc-channel.c b/src/ngircd/irc-channel.c index 1014d3a..138a94a 100644 --- a/src/ngircd/irc-channel.c +++ b/src/ngircd/irc-channel.c @@ -32,7 +32,6 @@ #include "parse.h" #include "irc-info.h" #include "irc-write.h" -#include "resolve.h" #include "conf.h" #include "exp.h" diff --git a/src/ngircd/irc-info.c b/src/ngircd/irc-info.c index 7e0bf63..806ab27 100644 --- a/src/ngircd/irc-info.c +++ b/src/ngircd/irc-info.c @@ -27,7 +27,6 @@ #include "conn-zip.h" #include "client.h" #include "channel.h" -#include "resolve.h" #include "conf.h" #include "defines.h" #include "log.h" diff --git a/src/ngircd/irc-login.c b/src/ngircd/irc-login.c index 8e77e7f..cd1899c 100644 --- a/src/ngircd/irc-login.c +++ b/src/ngircd/irc-login.c @@ -22,7 +22,6 @@ #include <strings.h> #include "ngircd.h" -#include "resolve.h" #include "conn-func.h" #include "conf.h" #include "client.h" diff --git a/src/ngircd/irc-mode.c b/src/ngircd/irc-mode.c index 10d4641..9a0f397 100644 --- a/src/ngircd/irc-mode.c +++ b/src/ngircd/irc-mode.c @@ -29,7 +29,6 @@ #include "log.h" #include "parse.h" #include "messages.h" -#include "resolve.h" #include "conf.h" #include "exp.h" diff --git a/src/ngircd/irc-oper.c b/src/ngircd/irc-oper.c index 8492603..900c860 100644 --- a/src/ngircd/irc-oper.c +++ b/src/ngircd/irc-oper.c @@ -21,7 +21,6 @@ #include <string.h> #include "ngircd.h" -#include "resolve.h" #include "conn-func.h" #include "conf.h" #include "client.h" diff --git a/src/ngircd/irc-server.c b/src/ngircd/irc-server.c index a0a9468..987513b 100644 --- a/src/ngircd/irc-server.c +++ b/src/ngircd/irc-server.c @@ -22,7 +22,6 @@ #include <strings.h> #include "defines.h" -#include "resolve.h" #include "conn.h" #include "conn-func.h" #include "conn-zip.h" diff --git a/src/ngircd/irc.c b/src/ngircd/irc.c index 0cb9a6e..f913d8d 100644 --- a/src/ngircd/irc.c +++ b/src/ngircd/irc.c @@ -22,7 +22,6 @@ static char UNUSED id[] = "$Id: irc.c,v 1.132 2008/01/15 22:28:14 fw Exp $"; #include <string.h> #include "ngircd.h" -#include "resolve.h" #include "conn-func.h" #include "conf.h" #include "client.h" diff --git a/src/ngircd/ngircd.c b/src/ngircd/ngircd.c index b951bad..8b2e4a8 100644 --- a/src/ngircd/ngircd.c +++ b/src/ngircd/ngircd.c @@ -39,7 +39,6 @@ #endif #include "defines.h" -#include "resolve.h" #include "conn.h" #include "conf-ssl.h" #include "client.h" diff --git a/src/ngircd/numeric.c b/src/ngircd/numeric.c index c3d94c3..b0a7798 100644 --- a/src/ngircd/numeric.c +++ b/src/ngircd/numeric.c @@ -20,7 +20,6 @@ #include <string.h> #include "defines.h" -#include "resolve.h" #include "conn.h" #include "conf.h" #include "conn.h"

1 0

Alexander Barton : New "module" proc.c/proc.h for generic process handling
by alex＠arthur.barton.de 14 Jul '10

14 Jul '10

Module: ngircd.git Branch: master Commit: 54e67ea9ee6c2b00c43f759edc55b57b969c9e2d URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=54e67ea9… Author: Alexander Barton <alex(a)barton.de> Date: Tue May 18 17:38:12 2010 +0200 New "module" proc.c/proc.h for generic process handling The new "module" proc.c is used for functions dealing with child processes. At the moment, it is only used by the asynchronous resolver. All the functions already implemented habe been migrated from the resolver code base, and the rest of the ngIRCd source code has been adepted to the new namespace and calling conventions. The goal is to develop "generic" process handling functions that can be used for other purposes as well, e.g. running processes on client connects etc. --- contrib/MacOSX/ngIRCd.xcodeproj/project.pbxproj | 6 ++ src/ngircd/Makefile.am | 10 +-- src/ngircd/conf.c | 2 +- src/ngircd/conf.h | 4 +- src/ngircd/conn.c | 23 +++-- src/ngircd/conn.h | 4 +- src/ngircd/io.c | 3 +- src/ngircd/proc.c | 102 +++++++++++++++++++++++ src/ngircd/proc.h | 35 ++++++++ src/ngircd/resolve.c | 99 +++------------------- src/ngircd/resolve.h | 30 ++----- 11 files changed, 187 insertions(+), 131 deletions(-) Diff: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commitdiff;h=54e6…

1 0

Alexander Barton : Refactoring: Rename CONNECTION.res_stat to .proc_stat
by alex＠arthur.barton.de 14 Jul '10

14 Jul '10

Module: ngircd.git Branch: master Commit: 89e73ad4b4e66921fc20c5afbe17042aee77a329 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=89e73ad4… Author: Alexander Barton <alex(a)barton.de> Date: Fri May 21 22:48:34 2010 +0200 Refactoring: Rename CONNECTION.res_stat to .proc_stat We want to use this process status variable not only for the resolver subprocesses but other asynchronous tasks as well; so let's name it more generic. --- src/ngircd/conn.c | 14 +++++++------- src/ngircd/conn.h | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c index 495c54f..91ff779 100644 --- a/src/ngircd/conn.c +++ b/src/ngircd/conn.c @@ -755,7 +755,7 @@ Conn_Handler(void) if (SSL_WantWrite(&My_Connections[i])) continue; /* TLS/SSL layer needs to write data; deal with this first */ #endif - if (Proc_InProgress(&My_Connections[i].res_stat)) { + if (Proc_InProgress(&My_Connections[i].proc_stat)) { /* Wait for completion of resolver sub-process ... */ io_event_del(My_Connections[i].sock, IO_WANTREAD); @@ -1074,8 +1074,8 @@ Conn_Close( CONN_ID Idx, const char *LogMsg, const char *FwdMsg, bool InformClie } /* cancel running resolver */ - if (Proc_InProgress(&My_Connections[Idx].res_stat)) - Proc_Kill(&My_Connections[Idx].res_stat); + if (Proc_InProgress(&My_Connections[Idx].proc_stat)) + Proc_Kill(&My_Connections[Idx].proc_stat); /* Servers: Modify time of next connect attempt? */ Conf_UnsetServer( Idx ); @@ -1377,7 +1377,7 @@ New_Connection(int Sock) identsock = -1; #endif if (!Conf_NoDNS) - Resolve_Addr(&My_Connections[new_sock].res_stat, &new_addr, + Resolve_Addr(&My_Connections[new_sock].proc_stat, &new_addr, identsock, cb_Read_Resolver_Result); /* ngIRCd waits up to 4 seconds for the result of the asynchronous @@ -1889,7 +1889,7 @@ Init_Conn_Struct(CONN_ID Idx) My_Connections[Idx].signon = now; My_Connections[Idx].lastdata = now; My_Connections[Idx].lastprivmsg = now; - Proc_InitStruct(&My_Connections[Idx].res_stat); + Proc_InitStruct(&My_Connections[Idx].proc_stat); } /* Init_Conn_Struct */ @@ -2003,7 +2003,7 @@ cb_Read_Resolver_Result( int r_fd, UNUSED short events ) /* Search associated connection ... */ for( i = 0; i < Pool_Size; i++ ) { if(( My_Connections[i].sock != NONE ) - && (Proc_GetPipeFd(&My_Connections[i].res_stat) == r_fd)) + && (Proc_GetPipeFd(&My_Connections[i].proc_stat) == r_fd)) break; } if( i >= Pool_Size ) { @@ -2015,7 +2015,7 @@ cb_Read_Resolver_Result( int r_fd, UNUSED short events ) } /* Read result from pipe */ - len = Resolve_Read(&My_Connections[i].res_stat, readbuf, sizeof readbuf -1); + len = Resolve_Read(&My_Connections[i].proc_stat, readbuf, sizeof readbuf -1); if (len == 0) return; diff --git a/src/ngircd/conn.h b/src/ngircd/conn.h index 2ca0d62..7665bb3 100644 --- a/src/ngircd/conn.h +++ b/src/ngircd/conn.h @@ -69,7 +69,7 @@ typedef struct _Connection { int sock; /* Socket handle */ ng_ipaddr_t addr; /* Client address */ - PROC_STAT res_stat; /* Status of resolver process */ + PROC_STAT proc_stat; /* Status of resolver process */ char host[HOST_LEN]; /* Hostname */ array rbuf; /* Read buffer */ array wbuf; /* Write buffer */

1 0

Alexander Barton : Update comments: subprocesses not only can be resolver processes
by alex＠arthur.barton.de 14 Jul '10

14 Jul '10

Module: ngircd.git Branch: master Commit: 60f5dd5b29e701b3ab601addd3cdefb508acc7d1 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=60f5dd5b… Author: Alexander Barton <alex(a)barton.de> Date: Fri May 21 23:44:20 2010 +0200 Update comments: subprocesses not only can be resolver processes --- src/ngircd/conn.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c index 91ff779..dff9336 100644 --- a/src/ngircd/conn.c +++ b/src/ngircd/conn.c @@ -756,7 +756,8 @@ Conn_Handler(void) continue; /* TLS/SSL layer needs to write data; deal with this first */ #endif if (Proc_InProgress(&My_Connections[i].proc_stat)) { - /* Wait for completion of resolver sub-process ... */ + /* Wait for completion of forked subprocess + * and ignore the socket in the meantime ... */ io_event_del(My_Connections[i].sock, IO_WANTREAD); continue; @@ -772,6 +773,7 @@ Conn_Handler(void) IO_WANTREAD); continue; } + io_event_add(My_Connections[i].sock, IO_WANTREAD); } @@ -1073,7 +1075,7 @@ Conn_Close( CONN_ID Idx, const char *LogMsg, const char *FwdMsg, bool InformClie in_k, out_k); } - /* cancel running resolver */ + /* Kill possibly running subprocess */ if (Proc_InProgress(&My_Connections[Idx].proc_stat)) Proc_Kill(&My_Connections[Idx].proc_stat);

1 0

Alexander Barton : Fix redundant redeclaration of Conn_Count*() functions
by alex＠arthur.barton.de 26 Jun '10

26 Jun '10

Module: ngircd.git Branch: master Commit: ae55d4f500703114cf10d07be786f1108d66b27d URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=ae55d4f5… Author: Alexander Barton <alex(a)barton.de> Date: Sat Jun 26 00:45:11 2010 +0200 Fix redundant redeclaration of Conn_Count*() functions The wrongly placed #endif lead to the following compiler warnings: conn.h:125: warning: redundant redeclaration of ‘Conn_Count’ conn.h:125: warning: previous declaration of ‘Conn_Count’ was here conn.h:126: warning: redundant redeclaration of ‘Conn_CountMax’ conn.h:126: warning: previous declaration of ‘Conn_CountMax’ was here conn.h:127: warning: redundant redeclaration of ‘Conn_CountAccepted’ conn.h:127: warning: previous declaration of ‘Conn_CountAccepted’ was here --- src/ngircd/conn.h | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/ngircd/conn.h b/src/ngircd/conn.h index 1cfa466..96d1f33 100644 --- a/src/ngircd/conn.h +++ b/src/ngircd/conn.h @@ -120,11 +120,12 @@ GLOBAL bool Conn_UsesSSL PARAMS((CONN_ID Idx)); #else static inline bool Conn_UsesSSL(UNUSED CONN_ID Idx) { return false; } #endif -#endif GLOBAL long Conn_Count PARAMS((void)); GLOBAL long Conn_CountMax PARAMS((void)); GLOBAL long Conn_CountAccepted PARAMS((void)); +#endif + /* -eof- */

1 0

Alexander Barton : const'ify Client_TypeText()
by alex＠arthur.barton.de 26 Jun '10

26 Jun '10

Module: ngircd.git Branch: master Commit: a68103771ccdf041c426277bc3a1f8dc345f3642 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=a6810377… Author: Alexander Barton <alex(a)barton.de> Date: Sat Jun 26 00:31:08 2010 +0200 const'ify Client_TypeText() --- src/ngircd/client.c | 2 +- src/ngircd/client.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ngircd/client.c b/src/ngircd/client.c index 671bf6c..7cf10da 100644 --- a/src/ngircd/client.c +++ b/src/ngircd/client.c @@ -1104,7 +1104,7 @@ Client_RegisterWhowas( CLIENT *Client ) } /* Client_RegisterWhowas */ -GLOBAL char * +GLOBAL const char * Client_TypeText(CLIENT *Client) { assert(Client != NULL); diff --git a/src/ngircd/client.h b/src/ngircd/client.h index 1064917..90fd6f2 100644 --- a/src/ngircd/client.h +++ b/src/ngircd/client.h @@ -146,7 +146,7 @@ GLOBAL int Client_GetLastWhowasIndex PARAMS(( void )); GLOBAL void Client_RegisterWhowas PARAMS(( CLIENT *Client )); -GLOBAL char * Client_TypeText PARAMS((CLIENT *Client)); +GLOBAL const char *Client_TypeText PARAMS((CLIENT *Client)); #endif

1 0

Alexander Barton : const'ify command name variable in _COMMAND strcuture
by alex＠arthur.barton.de 26 Jun '10

26 Jun '10

Module: ngircd.git Branch: master Commit: 8605e9c0fe7ffa42149271c9af31288bd4a0dfac URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=8605e9c0… Author: Alexander Barton <alex(a)barton.de> Date: Sat Jun 26 00:37:06 2010 +0200 const'ify command name variable in _COMMAND strcuture --- src/ngircd/parse.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/ngircd/parse.h b/src/ngircd/parse.h index a3f764e..205dbb8 100644 --- a/src/ngircd/parse.h +++ b/src/ngircd/parse.h @@ -30,7 +30,7 @@ typedef struct _REQUEST /* vgl. RFC 2812, 2.3 */ typedef struct _COMMAND { - char *name; /* command name */ + const char *name; /* command name */ bool (*function) PARAMS(( CLIENT *Client, REQUEST *Request )); CLIENT_TYPE type; /* valid client types (bit mask) */ long lcount, rcount; /* number of local and remote calls */

1 0

Alexander Barton : const'ify IRC_WriteStrXXX() and Get_Prefix() functions
by alex＠arthur.barton.de 26 Jun '10

26 Jun '10

Module: ngircd.git Branch: master Commit: 0c0d4af55ae6d098ccfabc258508a6b85b8c7449 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=0c0d4af5… Author: Alexander Barton <alex(a)barton.de> Date: Sat Jun 26 00:38:20 2010 +0200 const'ify IRC_WriteStrXXX() and Get_Prefix() functions --- src/ngircd/irc-write.c | 57 ++++++++++++++++++++++++++--------------------- src/ngircd/irc-write.h | 17 +++++++------ 2 files changed, 40 insertions(+), 34 deletions(-) diff --git a/src/ngircd/irc-write.c b/src/ngircd/irc-write.c index 9b5b5e2..afefefb 100644 --- a/src/ngircd/irc-write.c +++ b/src/ngircd/irc-write.c @@ -37,19 +37,19 @@ #define SEND_TO_SERVER 2 -static char *Get_Prefix PARAMS((CLIENT *Target, CLIENT *Client)); +static const char *Get_Prefix PARAMS((CLIENT *Target, CLIENT *Client)); static void cb_writeStrServersPrefixFlag PARAMS((CLIENT *Client, CLIENT *Prefix, void *Buffer)); #ifdef PROTOTYPES GLOBAL bool -IRC_WriteStrClient( CLIENT *Client, char *Format, ... ) +IRC_WriteStrClient( CLIENT *Client, const char *Format, ... ) #else GLOBAL bool IRC_WriteStrClient( Client, Format, va_alist ) CLIENT *Client; -char *Format; +const char *Format; va_dcl #endif { @@ -77,13 +77,13 @@ va_dcl #ifdef PROTOTYPES GLOBAL bool -IRC_WriteStrClientPrefix( CLIENT *Client, CLIENT *Prefix, char *Format, ... ) +IRC_WriteStrClientPrefix(CLIENT *Client, CLIENT *Prefix, const char *Format, ...) #else GLOBAL bool -IRC_WriteStrClientPrefix( Client, Prefix, Format, va_alist ) +IRC_WriteStrClientPrefix(Client, Prefix, Format, va_alist) CLIENT *Client; CLIENT *Prefix; -char *Format; +const char *Format; va_dcl #endif { @@ -110,14 +110,15 @@ va_dcl #ifdef PROTOTYPES GLOBAL bool -IRC_WriteStrChannel( CLIENT *Client, CHANNEL *Chan, bool Remote, char *Format, ... ) +IRC_WriteStrChannel(CLIENT *Client, CHANNEL *Chan, bool Remote, + const char *Format, ...) #else GLOBAL bool -IRC_WriteStrChannel( Client, Chan, Remote, Format, va_alist ) +IRC_WriteStrChannel(Client, Chan, Remote, Format, va_alist) CLIENT *Client; CHANNEL *Chan; bool Remote; -char *Format; +const char *Format; va_dcl #endif { @@ -146,15 +147,16 @@ va_dcl */ #ifdef PROTOTYPES GLOBAL bool -IRC_WriteStrChannelPrefix( CLIENT *Client, CHANNEL *Chan, CLIENT *Prefix, bool Remote, char *Format, ... ) +IRC_WriteStrChannelPrefix(CLIENT *Client, CHANNEL *Chan, CLIENT *Prefix, + bool Remote, const char *Format, ...) #else GLOBAL bool -IRC_WriteStrChannelPrefix( Client, Chan, Prefix, Remote, Format, va_alist ) +IRC_WriteStrChannelPrefix(Client, Chan, Prefix, Remote, Format, va_alist) CLIENT *Client; CHANNEL *Chan; CLIENT *Prefix; bool Remote; -char *Format; +const char *Format; va_dcl #endif { @@ -218,12 +220,12 @@ va_dcl #ifdef PROTOTYPES GLOBAL void -IRC_WriteStrServers( CLIENT *ExceptOf, char *Format, ... ) +IRC_WriteStrServers(CLIENT *ExceptOf, const char *Format, ...) #else GLOBAL void -IRC_WriteStrServers( ExceptOf, Format, va_alist ) +IRC_WriteStrServers(ExceptOf, Format, va_alist) CLIENT *ExceptOf; -char *Format; +const char *Format; va_dcl #endif { @@ -246,13 +248,14 @@ va_dcl #ifdef PROTOTYPES GLOBAL void -IRC_WriteStrServersPrefix( CLIENT *ExceptOf, CLIENT *Prefix, char *Format, ... ) +IRC_WriteStrServersPrefix(CLIENT *ExceptOf, CLIENT *Prefix, + const char *Format, ...) #else GLOBAL void -IRC_WriteStrServersPrefix( ExceptOf, Prefix, Format, va_alist ) +IRC_WriteStrServersPrefix(ExceptOf, Prefix, Format, va_alist) CLIENT *ExceptOf; CLIENT *Prefix; -char *Format; +const char *Format; va_dcl #endif { @@ -276,14 +279,15 @@ va_dcl #ifdef PROTOTYPES GLOBAL void -IRC_WriteStrServersPrefixFlag( CLIENT *ExceptOf, CLIENT *Prefix, char Flag, char *Format, ... ) +IRC_WriteStrServersPrefixFlag(CLIENT *ExceptOf, CLIENT *Prefix, char Flag, + const char *Format, ...) #else GLOBAL void -IRC_WriteStrServersPrefixFlag( ExceptOf, Prefix, Flag, Format, va_alist ) +IRC_WriteStrServersPrefixFlag(ExceptOf, Prefix, Flag, Format, va_alist) CLIENT *ExceptOf; CLIENT *Prefix; char Flag; -char *Format; +const char *Format; va_dcl #endif { @@ -331,14 +335,15 @@ IRC_WriteStrServersPrefixFlag_CB(CLIENT *ExceptOf, CLIENT *Prefix, char Flag, */ #ifdef PROTOTYPES GLOBAL bool -IRC_WriteStrRelatedPrefix( CLIENT *Client, CLIENT *Prefix, bool Remote, char *Format, ... ) +IRC_WriteStrRelatedPrefix(CLIENT *Client, CLIENT *Prefix, bool Remote, + const char *Format, ...) #else GLOBAL bool -IRC_WriteStrRelatedPrefix( Client, Prefix, Remote, Format, va_alist ) +IRC_WriteStrRelatedPrefix(Client, Prefix, Remote, Format, va_alist) CLIENT *Client; CLIENT *Prefix; bool Remote; -char *Format; +const char *Format; va_dcl #endif { @@ -416,7 +421,7 @@ GLOBAL void IRC_SendWallops(Client, From, Format, va_alist ) CLIENT *Client; CLIENT *From; -char *Format; +const char *Format; va_dcl #endif { @@ -468,7 +473,7 @@ IRC_SetPenalty( CLIENT *Client, time_t Seconds ) } /* IRC_SetPenalty */ -static char * +static const char * Get_Prefix( CLIENT *Target, CLIENT *Client ) { assert( Target != NULL ); diff --git a/src/ngircd/irc-write.h b/src/ngircd/irc-write.h index a984997..f054314 100644 --- a/src/ngircd/irc-write.h +++ b/src/ngircd/irc-write.h @@ -14,26 +14,27 @@ #ifndef __irc_write_h__ #define __irc_write_h__ -GLOBAL bool IRC_WriteStrClient PARAMS((CLIENT *Client, char *Format, ...)); +GLOBAL bool IRC_WriteStrClient PARAMS((CLIENT *Client, const char *Format, ...)); GLOBAL bool IRC_WriteStrClientPrefix PARAMS((CLIENT *Client, CLIENT *Prefix, - char *Format, ...)); + const char *Format, ...)); GLOBAL bool IRC_WriteStrChannel PARAMS((CLIENT *Client, CHANNEL *Chan, - bool Remote, char *Format, ...)); + bool Remote, const char *Format, ...)); GLOBAL bool IRC_WriteStrChannelPrefix PARAMS((CLIENT *Client, CHANNEL *Chan, - CLIENT *Prefix, bool Remote, char *Format, ...)); + CLIENT *Prefix, bool Remote, const char *Format, ...)); -GLOBAL void IRC_WriteStrServers PARAMS((CLIENT *ExceptOf, char *Format, ...)); +GLOBAL void IRC_WriteStrServers PARAMS((CLIENT *ExceptOf, + const char *Format, ...)); GLOBAL void IRC_WriteStrServersPrefix PARAMS((CLIENT *ExceptOf, CLIENT *Prefix, - char *Format, ...)); + const char *Format, ...)); GLOBAL void IRC_WriteStrServersPrefixFlag PARAMS((CLIENT *ExceptOf, - CLIENT *Prefix, char Flag, char *Format, ...)); + CLIENT *Prefix, char Flag, const char *Format, ...)); GLOBAL void IRC_WriteStrServersPrefixFlag_CB PARAMS((CLIENT *ExceptOf, CLIENT *Prefix, char Flag, void (*callback)(CLIENT *, CLIENT *, void *), void *cb_data)); GLOBAL bool IRC_WriteStrRelatedPrefix PARAMS((CLIENT *Client, CLIENT *Prefix, - bool Remote, char *Format, ...)); + bool Remote, const char *Format, ...)); GLOBAL void IRC_SendWallops PARAMS((CLIENT *Client, CLIENT *From, const char *Format, ...));

1 0

Alexander Barton : const'ify Send_ListChange() function in irc-mode.c
by alex＠arthur.barton.de 26 Jun '10

26 Jun '10

Module: ngircd.git Branch: master Commit: c6742192a6d64d0fd74a23f49dca7cdbf72037ac URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=c6742192… Author: Alexander Barton <alex(a)barton.de> Date: Sat Jun 26 00:42:12 2010 +0200 const'ify Send_ListChange() function in irc-mode.c --- src/ngircd/irc-mode.c | 18 ++++++++++++------ 1 files changed, 12 insertions(+), 6 deletions(-) diff --git a/src/ngircd/irc-mode.c b/src/ngircd/irc-mode.c index cd47086..10d4641 100644 --- a/src/ngircd/irc-mode.c +++ b/src/ngircd/irc-mode.c @@ -36,13 +36,18 @@ #include "irc-mode.h" -static bool Client_Mode PARAMS(( CLIENT *Client, REQUEST *Req, CLIENT *Origin, CLIENT *Target )); -static bool Channel_Mode PARAMS(( CLIENT *Client, REQUEST *Req, CLIENT *Origin, CHANNEL *Channel )); +static bool Client_Mode PARAMS(( CLIENT *Client, REQUEST *Req, CLIENT *Origin, + CLIENT *Target )); +static bool Channel_Mode PARAMS(( CLIENT *Client, REQUEST *Req, CLIENT *Origin, + CHANNEL *Channel )); -static bool Add_Ban_Invite PARAMS((int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const char *Pattern)); -static bool Del_Ban_Invite PARAMS((int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const char *Pattern)); +static bool Add_Ban_Invite PARAMS((int what, CLIENT *Prefix, CLIENT *Client, + CHANNEL *Channel, const char *Pattern)); +static bool Del_Ban_Invite PARAMS((int what, CLIENT *Prefix, CLIENT *Client, + CHANNEL *Channel, const char *Pattern)); -static bool Send_ListChange PARAMS(( char *Mode, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const char *Mask )); +static bool Send_ListChange PARAMS((const char *Mode, CLIENT *Prefix, + CLIENT *Client, CHANNEL *Channel, const char *Mask)); GLOBAL bool @@ -758,7 +763,8 @@ Del_Ban_Invite(int what, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const static bool -Send_ListChange( char *Mode, CLIENT *Prefix, CLIENT *Client, CHANNEL *Channel, const char *Mask ) +Send_ListChange(const char *Mode, CLIENT *Prefix, CLIENT *Client, + CHANNEL *Channel, const char *Mask) { bool ok;

1 0

Alexander Barton : const'ify Conn_WriteStr() function
by alex＠arthur.barton.de 26 Jun '10

26 Jun '10

Module: ngircd.git Branch: master Commit: edfa2154816932ce184f9db65093711bc9957cb3 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=edfa2154… Author: Alexander Barton <alex(a)barton.de> Date: Sat Jun 26 00:44:37 2010 +0200 const'ify Conn_WriteStr() function --- src/ngircd/conn.c | 6 +++--- src/ngircd/conn.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c index 6994c8b..bfcb8d4 100644 --- a/src/ngircd/conn.c +++ b/src/ngircd/conn.c @@ -809,12 +809,12 @@ Conn_Handler(void) */ #ifdef PROTOTYPES GLOBAL bool -Conn_WriteStr( CONN_ID Idx, char *Format, ... ) +Conn_WriteStr(CONN_ID Idx, const char *Format, ...) #else GLOBAL bool -Conn_WriteStr( Idx, Format, va_alist ) +Conn_WriteStr(Idx, Format, va_alist) CONN_ID Idx; -char *Format; +const char *Format; va_dcl #endif { diff --git a/src/ngircd/conn.h b/src/ngircd/conn.h index f2ec823..1cfa466 100644 --- a/src/ngircd/conn.h +++ b/src/ngircd/conn.h @@ -107,7 +107,7 @@ GLOBAL void Conn_ExitListeners PARAMS(( void )); GLOBAL void Conn_Handler PARAMS(( void )); -GLOBAL bool Conn_WriteStr PARAMS(( CONN_ID Idx, char *Format, ... )); +GLOBAL bool Conn_WriteStr PARAMS(( CONN_ID Idx, const char *Format, ... )); GLOBAL void Conn_Close PARAMS(( CONN_ID Idx, const char *LogMsg, const char *FwdMsg, bool InformClient ));

1 0