From c86d228e07fbc51e0704f6285636fd8750465d2e Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C4=81nis=20R=C5=ABcis?= Date: Tue, 30 Mar 2010 18:33:46 +0300 Subject: [PATCH] IRC: factor out message parsing --- irc.c | 90 ++++++++++++++++++++++++++++++++++++++++------------------- 1 file changed, 61 insertions(+), 29 deletions(-) diff --git a/irc.c b/irc.c index 85c4ccd2..b2b41a7b 100644 --- a/irc.c +++ b/irc.c @@ -123,14 +123,39 @@ typedef struct ircmessage_s } ircmessage_t; -static void IRC_ProcessMessage(const char *line) +static ircmessage_t *IRC_AllocMessage(void) +{ + ircmessage_t *msg; + + if ((msg = Z_Malloc(sizeof (*msg)))) + memset(msg, 0, sizeof (*msg)); + + return msg; +} + +static void IRC_FreeMessage(ircmessage_t *msg) +{ + if (msg) + { + if (msg->prefix) Z_Free(msg->prefix); + if (msg->command) Z_Free(msg->command); + if (msg->args) Z_Free(msg->args); + + Z_Free(msg); + } +} + +static ircmessage_t *IRC_ParseMessage(const char *line) { const int line_len = strlen(line); const char *line_end = line + line_len; - ircmessage_t msg; + ircmessage_t *msg; int len; - memset(&msg, 0, sizeof (msg)); + msg = IRC_AllocMessage(); + + if (!msg) + return NULL; if (line[0] == ':') { @@ -138,24 +163,29 @@ static void IRC_ProcessMessage(const char *line) len = strcspn(line, " "); + /* TODO, is this really needed? */ + if (line + len == line_end) - return; + { + IRC_FreeMessage(msg); + return NULL; + } - msg.prefix = Z_Malloc(len + 1); - memcpy(msg.prefix, line, len); - msg.prefix[len] = 0; + msg->prefix = Z_Malloc(len + 1); + memcpy(msg->prefix, line, len); + msg->prefix[len] = 0; line += len; line += strspn(line, " "); } else - msg.prefix = NULL; + msg->prefix = NULL; len = strcspn(line, " "); - msg.command = Z_Malloc(len + 1); - memcpy(msg.command, line, len); - msg.command[len] = 0; + msg->command = Z_Malloc(len + 1); + memcpy(msg->command, line, len); + msg->command[len] = 0; if (line + len != line_end) { @@ -164,29 +194,31 @@ static void IRC_ProcessMessage(const char *line) len = line_end - line; - msg.args = Z_Malloc(len + 1); - memcpy(msg.args, line, len); - msg.args[len] = 0; + msg->args = Z_Malloc(len + 1); + memcpy(msg->args, line, len); + msg->args[len] = 0; } else - msg.args = NULL; - - Con_Printf("[IRC] prefix: %s\n" - "[IRC] command: %s\n" - "[IRC] args: %s\n", - msg.prefix ? msg.prefix : "", - msg.command, - msg.args ? msg.args : ""); - - /* Free stuff. */ + msg->args = NULL; - if (msg.prefix) - Z_Free(msg.prefix); + return msg; +} - Z_Free(msg.command); +static void IRC_ProcessMessage(const char *line) +{ + ircmessage_t *msg; - if (msg.args) - Z_Free(msg.args); + if ((msg = IRC_ParseMessage(line))) + { + Con_Printf("[IRC] prefix: %s\n" + "[IRC] command: %s\n" + "[IRC] args: %s\n", + msg->prefix ? msg->prefix : "", + msg->command, + msg->args ? msg->args : ""); + + IRC_FreeMessage(msg); + } } static void IRC_ProcessAllMessages(void) -- 2.39.2