Changeset 830864d for irc_channel.c

Timestamp:

2015-03-15T09:31:18Z (9 years ago)

Author:

dequis <dx@…>

Children:

ac6855b3

Parents:

c0e4c22

git-author:

dequis <dx@…> (05-03-15 10:31:24)

git-committer:

dequis <dx@…> (15-03-15 09:31:18)

Message:

WIP placeholder channels with hipchat implementation

i was going to clean this up and split in two commits but uhhh...
maybe some other day, i'm tired now

not very tested and i'm not 100% happy about the design, but sucks way
less than what i had in the hip-cat branch

feedback still appreciated.

this adds channels to the channel list without creating groupchats for
them, allowing users to /join them. what the hip-cat branch did before
but with proper api and hopefully less dumb behavior

it still 'leaks' them intentionally, just like it did before, but now it
prevents saving them to the xml so yay

also slightly improved channel name generation, refactored
bee_irc_chat_name_hint into three or four functions, and so on

File:

• irc_channel.c (modified) (2 diffs)

irc_channel.c

@@ -276 +276 @@
                 if (ic->irc->status & USTATUS_SHUTDOWN) {
                         /* Don't do anything fancy when we're shutting down anyway. */
-                } else if (ic->flags & IRC_CHANNEL_TEMP) {
+                } else if (ic->flags & IRC_CHANNEL_TEMP && !(ic->flags & IRC_CHANNEL_KEEP_PLACEHOLDER)) {
                         irc_channel_free_soon(ic);
                 } else {
@@ -556 +556 @@
 }
 
+gboolean irc_channel_is_unused(bee_t *bee, char *name)
+{
+        char *type, *chat_type;
+        irc_channel_t *oic;
+
+        if (!irc_channel_name_ok(name)) {
+                return FALSE;
+        }
+
+        if (!(oic = irc_channel_by_name(bee->ui_data, name))) {
+                return TRUE;
+        }
+
+        type = set_getstr(&oic->set, "type");
+        chat_type = set_getstr(&oic->set, "chat_type");
+
+        if (type && chat_type && oic->data == FALSE &&
+            strcmp(type, "chat") == 0 &&
+            strcmp(chat_type, "groupchat") == 0) {
+                /* There's a channel with this name already, but it looks
+                   like it's not in use yet. Most likely the IRC client
+                   rejoined the channel after a reconnect. Remove it so
+                   we can reuse its name. */
+                irc_channel_free(oic);
+                return TRUE;
+        }
+
+        return FALSE;
+}
+
+char *irc_channel_name_gen(bee_t *bee, const char *hint)
+{
+        char name[MAX_NICK_LENGTH + 1] = { 0 };
+
+        name[0] = '#';
+        strncpy(name + 1, hint, MAX_NICK_LENGTH - 1);
+        name[MAX_NICK_LENGTH] = '\0';
+
+        irc_channel_name_strip(name);
+
+        if (set_getbool(&bee->set, "lcnicks")) {
+                nick_lc(bee->ui_data, name + 1);
+        }
+
+        while (!irc_channel_is_unused(bee, name)) {
+                underscore_dedupe(name);
+        }
+
+        return g_strdup(name);
+}
+
+gboolean irc_channel_name_hint(irc_channel_t *ic, const char *name)
+{
+        irc_t *irc = ic->irc;
+        char *full_name;
+
+        /* Don't rename a channel if the user's in it already. */
+        if (ic->flags & IRC_CHANNEL_JOINED) {
+                return FALSE;
+        }
+
+        if (!(full_name = irc_channel_name_gen(irc->b, name))) {
+                return FALSE;
+        }
+
+        g_free(ic->name);
+        ic->name = full_name;
+
+        return TRUE;
+}
+
 static gint irc_channel_user_cmp(gconstpointer a_, gconstpointer b_)
 {