Changeset 7de07a0 for irc.c

Timestamp:

2018-07-12T08:42:45Z (6 years ago)

Author:

dequis <dx@…>

Parents:

b9d1fdc

git-author:

dequis <dx@…> (12-07-18 08:30:59)

git-committer:

dequis <dx@…> (12-07-18 08:42:45)

Message:

Use GString for irc->sendbuffer (simpler and faster)

The old code managed growing the buffer manually and called strlen()
very often. GString has a length field and the logic to grow the buffer
is none of our business. Probably smarter, too.

This takes a "blist all" of 14k users from 8.2 to 1.3 seconds.

File:

• irc.c (modified) (7 diffs)

irc.c

@@ -62 +62 @@
         irc->iconv = (GIConv) - 1;
         irc->oconv = (GIConv) - 1;
+
+        irc->sendbuffer = g_string_sized_new(IRC_MAX_LINE * 2);
 
         if (global.conf->ping_interval > 0 && global.conf->ping_timeout > 0) {
@@ -305 +307 @@
         }
 
-        g_free(irc->sendbuffer);
+        g_string_free(irc->sendbuffer, TRUE);
         g_free(irc->readbuffer);
         g_free(irc->password);
@@ -595 +597 @@
 
                 if (now) {
-                        g_free(irc->sendbuffer);
-                        irc->sendbuffer = g_strdup("\r\n");
+                        g_string_assign(irc->sendbuffer, "\r\n");
                 }
                 irc_vawrite(temp->data, format, params);
@@ -611 +612 @@
 void irc_vawrite(irc_t *irc, char *format, va_list params)
 {
-        int size;
         char line[IRC_MAX_LINE + 1];
 
@@ -638 +638 @@
         g_strlcat(line, "\r\n", IRC_MAX_LINE + 1);
 
-        if (irc->sendbuffer != NULL) {
-                gsize line_len = strlen(line);
-                gsize buffer_len = strlen(irc->sendbuffer);
-
-                size = buffer_len + line_len;
-                irc->sendbuffer = g_renew(char, irc->sendbuffer, size + 1);
-                strcpy((irc->sendbuffer + buffer_len), line);
-        } else {
-                irc->sendbuffer = g_strdup(line);
-        }
+        g_string_append(irc->sendbuffer, line);
 
         if (irc->w_watch_source_id == 0) {
@@ -669 +660 @@
 {
         ssize_t n;
-        size_t len;
-
-        if (irc->sendbuffer == NULL) {
+        size_t len = irc->sendbuffer->len;
+
+        if (len == 0) {
                 return;
         }
 
-        len = strlen(irc->sendbuffer);
-        if ((n = send(irc->fd, irc->sendbuffer, len, 0)) == len) {
-                g_free(irc->sendbuffer);
-                irc->sendbuffer = NULL;
+        if ((n = send(irc->fd, irc->sendbuffer->str, len, 0)) == len) {
+                g_string_truncate(irc->sendbuffer, 0);
 
                 b_event_remove(irc->w_watch_source_id);
                 irc->w_watch_source_id = 0;
         } else if (n > 0) {
-                char *s = g_strdup(irc->sendbuffer + n);
-                g_free(irc->sendbuffer);
-                irc->sendbuffer = s;
+                g_string_erase(irc->sendbuffer, 0, n);
         }
         /* Otherwise something went wrong and we don't currently care
@@ -702 +689 @@
                 b_event_remove(irc->w_watch_source_id);
                 irc->w_watch_source_id = 0;
-                g_free(irc->sendbuffer);
-                irc->sendbuffer = NULL;
+                g_string_truncate(irc->sendbuffer, 0);
         }