Changes in protocols/purple/purple.c [ba7618d:b38f655]

File:

• protocols/purple/purple.c (modified) (21 diffs)

protocols/purple/purple.c

@@ -113 +113 @@
            servers anyway! */
         if (!dir_fixed) {
-                PurpleCertificatePool *pool;
                 irc_t *irc = acc->bee->ui_data;
                 char *dir;
@@ -123 +122 @@
                 purple_blist_load();
                 purple_prefs_load();
-
-                if (proxytype == PROXY_SOCKS4A) {
-                        /* do this here after loading prefs. yes, i know, it sucks */
-                        purple_prefs_set_bool("/purple/proxy/socks4_remotedns", TRUE);
-                }
-
-                /* re-create the certificate cache directory */
-                pool = purple_certificate_find_pool("x509", "tls_peers");
-                dir = purple_certificate_pool_mkpath(pool, NULL);
-                purple_build_dir(dir, 0700);
-                g_free(dir);
-
                 dir_fixed = TRUE;
         }
@@ -360 +347 @@
         if (!pd) {
                 return;
-        }
-
-        while (ic->groupchats) {
-                imcb_chat_free(ic->groupchats->data);
         }
 
@@ -651 +634 @@
         /* Call the fucker. */
         callback = (void *) mi->callback;
-        callback(&pb->node, mi->data);
+        callback(&pb->node, menu->data);
 
         return NULL;
@@ -719 +702 @@
                         g_hash_table_replace(chat_hash, "passwd", g_strdup(password));
                 }
-
-                g_free(pce);
-        }
-
-        g_list_free(info);
+        }
 
         serv_join_chat(purple_account_get_connection(pd->account), chat_hash);
 
-        g_hash_table_destroy(chat_hash);
-
-        return imcb_chat_new(ic, room);
+        return NULL;
 }
 
@@ -751 +728 @@
 static PurpleCoreUiOps bee_core_uiops =
 {
-        NULL,                      /* ui_prefs_init */
-        NULL,                      /* debug_ui_init */
-        purple_ui_init,            /* ui_init */
-        NULL,                      /* quit */
-        prplcb_ui_info,            /* get_ui_info */
+        NULL,
+        NULL,
+        purple_ui_init,
+        NULL,
+        prplcb_ui_info,
 };
 
@@ -782 +759 @@
         purple_gg_buddylist_import(gc);
 
-        ic->flags |= OPT_DOES_HTML;
+        if (gc->flags & PURPLE_CONNECTION_HTML) {
+                ic->flags |= OPT_DOES_HTML;
+        }
 }
 
@@ -816 +795 @@
 static PurpleConnectionUiOps bee_conn_uiops =
 {
-        prplcb_conn_progress,                    /* connect_progress */
-        prplcb_conn_connected,                   /* connected */
-        prplcb_conn_disconnected,                /* disconnected */
-        prplcb_conn_notice,                      /* notice */
-        NULL,                                    /* report_disconnect */
-        NULL,                                    /* network_connected */
-        NULL,                                    /* network_disconnected */
-        prplcb_conn_report_disconnect_reason,    /* report_disconnect_reason */
+        prplcb_conn_progress,
+        prplcb_conn_connected,
+        prplcb_conn_disconnected,
+        prplcb_conn_notice,
+        NULL,
+        NULL,
+        NULL,
+        prplcb_conn_report_disconnect_reason,
 };
 
@@ -898 +877 @@
 static PurpleBlistUiOps bee_blist_uiops =
 {
-        NULL,                      /* new_list */
-        prplcb_blist_new,          /* new_node */
-        NULL,                      /* show */
-        prplcb_blist_update,       /* update */
-        prplcb_blist_remove,       /* remove */
+        NULL,
+        prplcb_blist_new,
+        NULL,
+        prplcb_blist_update,
+        prplcb_blist_remove,
 };
 
@@ -911 +890 @@
                 struct groupchat *gc;
 
-                gc = bee_chat_by_title(ic->bee, ic, conv->name);
-
-                if (!gc) {
-                        gc = imcb_chat_new(ic, conv->name);
-                        if (conv->title != NULL) {
-                                imcb_chat_name_hint(gc, conv->title);
-                        }
-                }
-
-                /* don't set the topic if it's just the name */
-                if (conv->title != NULL && strcmp(conv->name, conv->title) != 0) {
+                gc = imcb_chat_new(ic, conv->name);
+                if (conv->title != NULL) {
+                        imcb_chat_name_hint(gc, conv->title);
                         imcb_chat_topic(gc, NULL, conv->title, 0);
                 }
@@ -964 +935 @@
 }
 
-/* Generic handler for IM or chat messages, covers write_chat, write_im and write_conv */
-static void handle_conv_msg(PurpleConversation *conv, const char *who, const char *message, guint32 bee_flags, time_t mtime)
-{
-        struct im_connection *ic = purple_ic_by_pa(conv->account);
+void prplcb_conv_chat_msg(PurpleConversation *conv, const char *who, const char *message, PurpleMessageFlags flags,
+                          time_t mtime)
+{
         struct groupchat *gc = conv->ui_data;
         PurpleBuddy *buddy;
+
+        /* ..._SEND means it's an outgoing message, no need to echo those. */
+        if (flags & PURPLE_MESSAGE_SEND) {
+                return;
+        }
 
         buddy = purple_find_buddy(conv->account, who);
@@ -976 +951 @@
         }
 
-        if (conv->type == PURPLE_CONV_TYPE_IM) {
-                imcb_buddy_msg(ic, (char *) who, (char *) message, bee_flags, mtime);
-        } else if (gc) {
-                imcb_chat_msg(gc, who, (char *) message, bee_flags, mtime);
-        }
-}
-
-/* Handles write_im and write_chat. Removes echoes of locally sent messages */
-static void prplcb_conv_msg(PurpleConversation *conv, const char *who, const char *message, PurpleMessageFlags flags, time_t mtime)
-{
-        if (!(flags & PURPLE_MESSAGE_SEND)) {
-                handle_conv_msg(conv, who, message, 0, mtime);
-        }
-}
-
-/* Handles write_conv. Only passes self messages from other locations through.
- * That is, only writes of PURPLE_MESSAGE_SEND.
- * There are more events which might be handled in the future, but some are tricky.
- * (images look like <img id="123">, what do i do with that?) */
-static void prplcb_conv_write(PurpleConversation *conv, const char *who, const char *alias, const char *message,
-                              PurpleMessageFlags flags, time_t mtime)
-{
+        imcb_chat_msg(gc, who, (char *) message, 0, mtime);
+}
+
+static void prplcb_conv_im(PurpleConversation *conv, const char *who, const char *message, PurpleMessageFlags flags,
+                           time_t mtime)
+{
+        struct im_connection *ic = purple_ic_by_pa(conv->account);
+        PurpleBuddy *buddy;
+
+        /* ..._SEND means it's an outgoing message, no need to echo those. */
         if (flags & PURPLE_MESSAGE_SEND) {
-                handle_conv_msg(conv, who, message, OPT_SELFMESSAGE, mtime);
-        }
+                return;
+        }
+
+        buddy = purple_find_buddy(conv->account, who);
+        if (buddy != NULL) {
+                who = purple_buddy_get_name(buddy);
+        }
+
+        imcb_buddy_msg(ic, (char *) who, (char *) message, 0, mtime);
 }
 
@@ -1033 +1003 @@
         prplcb_conv_new,           /* create_conversation  */
         prplcb_conv_free,          /* destroy_conversation */
-        prplcb_conv_msg,           /* write_chat           */
-        prplcb_conv_msg,           /* write_im             */
-        prplcb_conv_write,         /* write_conv           */
+        prplcb_conv_chat_msg,      /* write_chat           */
+        prplcb_conv_im,            /* write_im             */
+        NULL,                      /* write_conv           */
         prplcb_conv_add_users,     /* chat_add_users       */
         NULL,                      /* chat_rename_user     */
@@ -1200 +1170 @@
 static PurpleRequestUiOps bee_request_uiops =
 {
-        prplcb_request_input,      /* request_input */
-        NULL,                      /* request_choice */
-        prplcb_request_action,     /* request_action */
-        NULL,                      /* request_fields */
-        NULL,                      /* request_file */
-        prplcb_close_request,      /* close_request */
-        NULL,                      /* request_folder */
+        prplcb_request_input,
+        NULL,
+        prplcb_request_action,
+        NULL,
+        NULL,
+        prplcb_close_request,
+        NULL,
 };
 
@@ -1247 +1217 @@
 static PurplePrivacyUiOps bee_privacy_uiops =
 {
-        prplcb_privacy_permit_added,       /* permit_added */
-        prplcb_privacy_permit_removed,     /* permit_removed */
-        prplcb_privacy_deny_added,         /* deny_added */
-        prplcb_privacy_deny_removed,       /* deny_removed */
+        prplcb_privacy_permit_added,
+        prplcb_privacy_permit_removed,
+        prplcb_privacy_deny_added,
+        prplcb_privacy_deny_removed,
 };
 
@@ -1260 +1230 @@
 static PurpleDebugUiOps bee_debug_uiops =
 {
-        prplcb_debug_print,        /* print */
+        prplcb_debug_print,
 };
 
@@ -1281 +1251 @@
 static PurpleEventLoopUiOps glib_eventloops =
 {
-        prplcb_ev_timeout_add,     /* timeout_add */
-        prplcb_ev_remove,          /* timeout_remove */
-        prplcb_ev_input_add,       /* input_add */
-        prplcb_ev_remove,          /* input_remove */
+        prplcb_ev_timeout_add,
+        prplcb_ev_remove,
+        prplcb_ev_input_add,
+        prplcb_ev_remove,
 };
-
-/* Absolutely no connection context at all. Thanks purple! brb crying */
-static void *prplcb_notify_message(PurpleNotifyMsgType type, const char *title,
-                                   const char *primary, const char *secondary)
-{
-        char *text = g_strdup_printf("%s%s - %s%s%s",
-                (type == PURPLE_NOTIFY_MSG_ERROR) ? "Error: " : "",
-                title,
-                primary ?: "",
-                (primary && secondary) ? " - " : "",
-                secondary ?: ""
-        );
-
-        if (local_bee->ui->log) {
-                local_bee->ui->log(local_bee, "purple", text);
-        }
-
-        g_free(text);
-
-        return NULL;
-}
 
 static void *prplcb_notify_email(PurpleConnection *gc, const char *subject, const char *from,
@@ -1368 +1317 @@
 static PurpleNotifyUiOps bee_notify_uiops =
 {
-        prplcb_notify_message,     /* notify_message */
-        prplcb_notify_email,       /* notify_email */
-        NULL,                      /* notify_emails */
-        NULL,                      /* notify_formatted */
-        NULL,                      /* notify_searchresults */
-        NULL,                      /* notify_searchresults_new_rows */
-        prplcb_notify_userinfo,    /* notify_userinfo */
+        NULL,
+        prplcb_notify_email,
+        NULL,
+        NULL,
+        NULL,
+        NULL,
+        prplcb_notify_userinfo,
 };
 
@@ -1401 +1350 @@
 static PurpleAccountUiOps bee_account_uiops =
 {
-        NULL,                              /* notify_added */
-        NULL,                              /* status_changed */
-        NULL,                              /* request_add */
-        prplcb_account_request_authorize,  /* request_authorize */
-        NULL,                              /* close_account_request */
+        NULL,
+        NULL,
+        NULL,
+        prplcb_account_request_authorize,
+        NULL,
 };
 
@@ -1433 +1382 @@
         char *dir;
 
-        g_assert((int) B_EV_IO_READ == (int) PURPLE_INPUT_READ);
-        g_assert((int) B_EV_IO_WRITE == (int) PURPLE_INPUT_WRITE);
+        if (B_EV_IO_READ != PURPLE_INPUT_READ ||
+            B_EV_IO_WRITE != PURPLE_INPUT_WRITE) {
+                /* FIXME FIXME FIXME FIXME FIXME :-) */
+                exit(1);
+        }
 
         dir = g_strdup_printf("%s/purple", global.conf->configdir);
@@ -1452 +1404 @@
                 PurpleProxyInfo *pi = purple_global_proxy_get_info();
                 switch (proxytype) {
-                case PROXY_SOCKS4A:
                 case PROXY_SOCKS4:
                         purple_proxy_info_set_type(pi, PURPLE_PROXY_SOCKS4);