]> jfr.im git - irc/irssi/irssi.git/commitdiff
core: remove unused len variable
authorEmil Engler <redacted>
Mon, 14 Aug 2023 10:01:31 +0000 (12:01 +0200)
committerEmil Engler <redacted>
Mon, 14 Aug 2023 10:01:31 +0000 (12:01 +0200)
This commit removes the unused `len` variable, which gets set quite a
few times, but whose value is totally unused.

This also fixes a compiler warning I get on my Darwin.

src/irc/core/netsplit.c

index 38b63fe53367af28998bb1cd14c1b0fbc52ffe00..308a20d99c63cec1965553d0e02ee4cf1dc0d33c 100644 (file)
@@ -222,7 +222,7 @@ NETSPLIT_CHAN_REC *netsplit_find_channel(IRC_SERVER_REC *server,
 int quitmsg_is_split(const char *msg)
 {
        const char *host2, *p;
-        int prev, len, host1_dot, host2_dot;
+        int prev, host1_dot, host2_dot;
 
        g_return_val_if_fail(msg != NULL, FALSE);
 
@@ -242,7 +242,7 @@ int quitmsg_is_split(const char *msg)
             - can't contain ':' or '/' chars (some servers allow URLs)
           */
        host2 = NULL;
-       prev = '\0'; len = 0; host1_dot = host2_dot = 0;
+       prev = '\0'; host1_dot = host2_dot = 0;
        while (*msg != '\0') {
                if (*msg == ' ') {
                        if (prev == '.' || prev == '\0') {
@@ -254,7 +254,7 @@ int quitmsg_is_split(const char *msg)
                                return FALSE; /* only one space allowed */
                        if (!host1_dot)
                                 return FALSE; /* host1 didn't have domain */
-                        host2 = msg+1; len = -1;
+                        host2 = msg+1;
                } else if (*msg == '.') {
                        if (prev == '\0' || prev == ' ' || prev == '.') {
                                /* domains can't start with '.'
@@ -270,7 +270,7 @@ int quitmsg_is_split(const char *msg)
                        return FALSE;
 
                prev = *msg;
-                msg++; len++;
+                msg++;
        }
 
        if (!host2_dot || prev == '.')