Changes in protocols/nogaim.c [eb73d05:5a8afc3]

File:

• protocols/nogaim.c (modified) (6 diffs)

protocols/nogaim.c

@@ -50 +50 @@
 }
 
-/* semi-private */
-gboolean plugin_info_validate(struct plugin_info *info, const char *path)
+gboolean load_plugin(char *path)
 {
         GList *l;
-        gboolean loaded = FALSE;
-
-        if (!path) {
-                path = "(null)";
-        }
-
-        if (info->abiver != BITLBEE_ABI_VERSION_CODE) {
-                log_message(LOGLVL_ERROR,
-                            "`%s' uses ABI %u but %u is required\n",
-                            path, info->abiver,
-                            BITLBEE_ABI_VERSION_CODE);
-                return FALSE;
-        }
-
-        if (!info->name || !info->version) {
-                log_message(LOGLVL_ERROR,
-                            "Name or version missing from the "
-                            "plugin info in `%s'\n", path);
-                return FALSE;
-        }
-
-        for (l = plugins; l; l = l->next) {
-                struct plugin_info *i = l->data;
-
-                if (g_strcasecmp(i->name, info->name) == 0) {
-                        loaded = TRUE;
-                        break;
-                }
-        }
-
-        if (loaded) {
-                log_message(LOGLVL_WARNING,
-                            "%s plugin already loaded\n",
-                            info->name);
-                return FALSE;
-        }
-
-        return TRUE;
-}
-
-/* semi-private */
-gboolean plugin_info_add(struct plugin_info *info)
-{
-        plugins = g_list_insert_sorted_with_data(plugins, info, pluginscmp, NULL);
-        return TRUE;
-}
-
-gboolean load_plugin(char *path)
-{
-        struct plugin_info *info = NULL;
+        struct plugin_info *i;
+        struct plugin_info *info;
         struct plugin_info * (*info_function) (void) = NULL;
         void (*init_function) (void);
 
         GModule *mod = g_module_open(path, G_MODULE_BIND_LAZY);
+        gboolean loaded = FALSE;
 
         if (!mod) {
@@ -117 +69 @@
                 info = info_function();
 
-                if (!plugin_info_validate(info, path)) {
+                if (info->abiver != BITLBEE_ABI_VERSION_CODE) {
+                        log_message(LOGLVL_ERROR,
+                                    "`%s' uses ABI %u but %u is required\n",
+                                    path, info->abiver,
+                                    BITLBEE_ABI_VERSION_CODE);
+                        g_module_close(mod);
+                        return FALSE;
+                }
+
+                if (!info->name || !info->version) {
+                        log_message(LOGLVL_ERROR,
+                                    "Name or version missing from the "
+                                    "plugin info in `%s'\n", path);
+                        g_module_close(mod);
+                        return FALSE;
+                }
+
+                for (l = plugins; l; l = l->next) {
+                        i = l->data;
+
+                        if (g_strcasecmp(i->name, info->name) == 0) {
+                                loaded = TRUE;
+                                break;
+                        }
+                }
+
+                if (loaded) {
+                        log_message(LOGLVL_WARNING,
+                                    "%s plugin already loaded\n",
+                                    info->name);
                         g_module_close(mod);
                         return FALSE;
@@ -132 +113 @@
 
         if (info_function) {
-                plugin_info_add(info);
+                plugins = g_list_insert_sorted_with_data(plugins, info,
+                                                         pluginscmp, NULL);
         }
 
@@ -768 +750 @@
                 msg = ic->acc->flags & ACC_FLAG_AWAY_MESSAGE ? away : NULL;
                 away = imc_away_state_find(m, away, &msg) ? :
-                       (imc_away_state_find(m, "away", NULL) ? : m->data);
+                       (imc_away_state_find(m, "away", &msg) ? : m->data);
         } else if (ic->acc->flags & ACC_FLAG_STATUS_MESSAGE) {
                 away = NULL;
@@ -802 +784 @@
                            contains no data unless it adds something to what
                            we have in state already. */
-                        if (message && strlen(m->data) == strlen(away)) {
+                        if (strlen(m->data) == strlen(away)) {
                                 *message = NULL;
                         }
@@ -828 +810 @@
                         for (m = gcm; m; m = m->next) {
                                 if (g_strcasecmp(imc_away_alias_list[i][j], m->data) == 0) {
-                                        if (!keep_message && message) {
+                                        if (!keep_message) {
                                                 *message = NULL;
                                         }