2005-07-20 [colin] 1.9.12cvs87
authorColin Leroy <colin@colino.net>
Wed, 20 Jul 2005 16:50:43 +0000 (16:50 +0000)
committerColin Leroy <colin@colino.net>
Wed, 20 Jul 2005 16:50:43 +0000 (16:50 +0000)
* src/folder.c
* src/textview.c
Gcc 2.9x compilation fixes, thanks to Darko
* src/plugins/clamav/clamav_plugin.c
Little optimisation: load virus database only once
* src/plugins/spamassassin/spamassassin.c
Big optimisation: filtering one mail took at least 1 second,
now it takes at least 50ms. Doesn't speed up spamassassin
itself, but should already be much more tolerable.

ChangeLog-gtk2.claws
PATCHSETS
configure.ac
src/folder.c
src/plugins/clamav/clamav_plugin.c
src/plugins/spamassassin/spamassassin.c
src/textview.c

index dd5caa7973c23f7ef60406aab439ba7fd1587de6..06ea27e4658a75093d742b759e30e22a4cf0ee93 100644 (file)
@@ -1,3 +1,15 @@
+2005-07-20 [colin]     1.9.12cvs87
+
+       * src/folder.c
+       * src/textview.c
+               Gcc 2.9x compilation fixes, thanks to Darko
+       * src/plugins/clamav/clamav_plugin.c
+               Little optimisation: load virus database only once
+       * src/plugins/spamassassin/spamassassin.c
+               Big optimisation: filtering one mail took at least 1 second,
+               now it takes at least 50ms. Doesn't speed up spamassassin 
+               itself, but should already be much more tolerable.
+
 2005-07-20 [colin]     1.9.12cvs86
 
        * src/summaryview.c
index 666b178dda2a32d0b38666a8227f5e22792b350f..f622e8de0bb4a1fd111e89b11fe200cee10eec32 100644 (file)
--- a/PATCHSETS
+++ b/PATCHSETS
 ( cvs diff -u -r 1.654.2.718 -r 1.654.2.719 configure.ac;  cvs diff -u -r 1.8.2.3 -r 1.8.2.4 src/plugins/Makefile.am;  diff -u /dev/null src/plugins/pgpinline/.cvsignore;  diff -u /dev/null src/plugins/pgpinline/Makefile.am;  diff -u /dev/null src/plugins/pgpinline/pgpinline.c;  diff -u /dev/null src/plugins/pgpinline/pgpinline.deps;  diff -u /dev/null src/plugins/pgpinline/pgpinline.h;  diff -u /dev/null src/plugins/pgpinline/plugin.c;  ) > 1.9.12cvs84.patchset
 ( cvs diff -u -r 1.1.2.1 -r 1.1.2.2 src/plugins/pgpinline/plugin.c;  ) > 1.9.12cvs85.patchset
 ( cvs diff -u -r 1.395.2.96 -r 1.395.2.97 src/summaryview.c;  ) > 1.9.12cvs86.patchset
+( cvs diff -u -r 1.213.2.41 -r 1.213.2.42 src/folder.c;  cvs diff -u -r 1.96.2.64 -r 1.96.2.65 src/textview.c;  cvs diff -u -r 1.13.2.6 -r 1.13.2.7 src/plugins/clamav/clamav_plugin.c;  cvs diff -u -r 1.18.2.7 -r 1.18.2.8 src/plugins/spamassassin/spamassassin.c;  ) > 1.9.12cvs87.patchset
index 3e002ac724c99322a653b7802a21faa45110c332..4194d35668a61209e4cdb13d3130c2acf89fdb5e 100644 (file)
@@ -11,7 +11,7 @@ MINOR_VERSION=9
 MICRO_VERSION=12
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=86
+EXTRA_VERSION=87
 EXTRA_RELEASE=
 EXTRA_GTK2_VERSION=
 
index 7ba9420b7261d4e0e31d05598948603a0cc0f56b..baf243d853b737a925a84fb429bd9618eca9b418 100644 (file)
@@ -2766,9 +2766,9 @@ static gint do_copy_msgs(FolderItem *dest, GSList *msglist, gboolean remove_sour
                                                                relation);
                }
                for (l = msglist; l != NULL; l = g_slist_next(l)) {
+                       GTuples *tuples;
                        msginfo = (MsgInfo *) l->data;
                        item = msginfo->folder;
-                       GTuples *tuples;
 
                        tuples = g_relation_select(relation, msginfo, 0);
                        num = GPOINTER_TO_INT(g_tuples_index(tuples, 0, 1));
index 482464b3982f5f5abec357f1cb3e7eedefe931bf..488fbaf9c24be1736b698cfc542d17f32f6c01a6 100644 (file)
@@ -60,11 +60,12 @@ static PrefParam param[] = {
        {NULL, NULL, NULL, P_OTHER, NULL, NULL, NULL}
 };
 
+static struct cl_node *cl_database;
 struct scan_parameters {
        gboolean is_infected;
 
-       struct cl_node *root;
        struct cl_limits limits;
+       struct cl_node *root;
        gboolean scan_archive;
 };
 
@@ -104,7 +105,6 @@ static gboolean mail_filtering_hook(gpointer source, gpointer data)
        MsgInfo *msginfo = mail_filtering_data->msginfo;
        MimeInfo *mimeinfo;
 
-       int ret, no = 0;
        struct scan_parameters params;
 
        if (!config.clamav_enable)
@@ -118,7 +118,7 @@ static gboolean mail_filtering_hook(gpointer source, gpointer data)
                message_callback(_("ClamAV: scanning message..."));
 
        params.is_infected = FALSE;
-       params.root = NULL;
+       params.root = cl_database;
 
        params.limits.maxfiles = 1000; /* max files */
        params.limits.maxfilesize = config.clamav_max_size * 1048576; /* maximum archived file size */
@@ -127,15 +127,6 @@ static gboolean mail_filtering_hook(gpointer source, gpointer data)
        if (config.clamav_enable_arc)
                params.scan_archive = TRUE;
 
-       if ((ret = cl_loaddbdir(cl_retdbdir(), &params.root, &no))) {
-               debug_print("cl_loaddbdir: %s\n", cl_strerror(ret));
-               return FALSE;
-       }
-
-       debug_print("Database loaded (containing in total %d signatures)\n", no);
-
-       cl_buildtrie(params.root);
-
        g_node_traverse(mimeinfo->node, G_PRE_ORDER, G_TRAVERSE_ALL, -1, scan_func, &params);
 
        if (params.is_infected) {
@@ -154,7 +145,6 @@ static gboolean mail_filtering_hook(gpointer source, gpointer data)
                }
        }
        
-       cl_freetrie(params.root);
        procmime_mimeinfo_free_all(mimeinfo);
        
        return params.is_infected;
@@ -196,7 +186,7 @@ void clamav_set_message_callback(MessageCallback callback)
 gint plugin_init(gchar **error)
 {
        gchar *rcpath;
-       
+       int no, ret;
        if ((sylpheed_get_version() > VERSION_NUMERIC)) {
                *error = g_strdup("Your sylpheed version is newer than the version the plugin was built with");
                return -1;
@@ -217,7 +207,16 @@ gint plugin_init(gchar **error)
        rcpath = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S, COMMON_RC, NULL);
        prefs_read_config(param, "ClamAV", rcpath, NULL);
        g_free(rcpath);
-       
+
+       if ((ret = cl_loaddbdir(cl_retdbdir(), &cl_database, &no)) != 0) {
+               debug_print("cl_loaddbdir: %s\n", cl_strerror(ret));
+               return -1;
+       }
+
+       debug_print("Database loaded (containing in total %d signatures)\n", no);
+
+       cl_buildtrie(cl_database);
+
        debug_print("ClamAV plugin loaded\n");
 
        return 0;
@@ -228,7 +227,7 @@ void plugin_done(void)
 {
        hooks_unregister_hook(MAIL_FILTERING_HOOKLIST, hook_id);
        g_free(config.clamav_save_folder);
-       
+       cl_freetrie(cl_database);
        debug_print("ClamAV plugin unloaded\n");
 }
 
index c35607158971bb08a44d16881f8aa06ebc0b3ad5..13e484fd96e9ad07df6cbe99ed3ebd713d58ade0 100644 (file)
@@ -195,7 +195,7 @@ static gboolean mail_filtering_hook(gpointer source, gpointer data)
 
                running |= CHILD_RUNNING;
 
-               g_timeout_add(1000, timeout_func, &running);
+               g_timeout_add(50, timeout_func, &running);
                running |= TIMEOUT_RUNNING;
 
                while(running & CHILD_RUNNING) {
index ac22ffec6c6e3236a2a2310a2503660cdc7c4d42..09c59dc3545e74cdefba161f0e0bc79f859a5967 100644 (file)
@@ -1160,11 +1160,6 @@ static void textview_make_clickable_parts(TextView *textview,
        GtkTextIter iter;
        gchar *mybuf = g_strdup(linebuf);
        
-       if (!g_utf8_validate(linebuf, -1, NULL)) {
-               mybuf = g_malloc(strlen(linebuf)*2 +1);
-               conv_localetodisp(mybuf, strlen(linebuf)*2 +1, linebuf);
-       }
-
        /* parse table - in order of priority */
        struct table {
                const gchar *needle; /* token */
@@ -1201,6 +1196,11 @@ static void textview_make_clickable_parts(TextView *textview,
                struct txtpos   *next;          /* next */
        } head = {NULL, NULL, 0,  NULL}, *last = &head;
 
+       if (!g_utf8_validate(linebuf, -1, NULL)) {
+               mybuf = g_malloc(strlen(linebuf)*2 +1);
+               conv_localetodisp(mybuf, strlen(linebuf)*2 +1, linebuf);
+       }
+
        gtk_text_buffer_get_end_iter(buffer, &iter);
 
        /* parse for clickable parts, and build a list of begin and end positions  */