From 344bb7b1bcd540e761e9a4e9589e910069403369 Mon Sep 17 00:00:00 2001 From: Radek Doulik Date: Wed, 17 Mar 2004 19:47:52 +0000 Subject: reverted last fix and: 2004-03-17 Radek Doulik * camel-folder-summary.h: use 1<<30 for CAMEL_MESSAGE_JUNK_LEARN, 1<<17 was already used by imap provider and maybe others svn path=/trunk/; revision=25103 --- camel/camel-folder.c | 59 ++++++++++++++++++++++++---------------------------- 1 file changed, 27 insertions(+), 32 deletions(-) (limited to 'camel/camel-folder.c') diff --git a/camel/camel-folder.c b/camel/camel-folder.c index 909a5e429c..620881b1fc 100644 --- a/camel/camel-folder.c +++ b/camel/camel-folder.c @@ -1714,42 +1714,37 @@ folder_changed (CamelObject *obj, gpointer event_data) camel_type_to_name (CAMEL_OBJECT_GET_TYPE (folder)))); return ret; } - - if ((folder->folder_flags & (CAMEL_FOLDER_FILTER_RECENT|CAMEL_FOLDER_FILTER_JUNK)) && changed->uid_recent->len > 0) { - const char *source; - - if (folder->folder_flags & CAMEL_FOLDER_FILTER_RECENT) - source = FILTER_SOURCE_INCOMING; - else - source = FILTER_SOURCE_JUNKTEST; - - if (folder->folder_flags & CAMEL_FOLDER_FILTER_JUNK) { - guint32 flags; - int i; - - for (i = 0; i < changed->uid_changed->len; i++) { - flags = camel_folder_get_message_flags (folder, changed->uid_changed->pdata[i]); - if (flags & CAMEL_MESSAGE_JUNK_LEARN) { - if (flags & CAMEL_MESSAGE_JUNK) { - if (!junk) - junk = g_ptr_array_new (); - g_ptr_array_add (junk, g_strdup (changed->uid_changed->pdata[i])); - } else { - if (!notjunk) - notjunk = g_ptr_array_new (); - g_ptr_array_add (notjunk, g_strdup (changed->uid_changed->pdata[i])); - } - - /* reset junk learn flag so that we don't process it again */ - camel_folder_set_message_flags (folder, changed->uid_changed->pdata[i], - CAMEL_MESSAGE_JUNK_LEARN, 0); + + if (changed->uid_changed->len) { + int i; + guint32 flags; + + for (i = 0; i < changed->uid_changed->len; i ++) { + flags = camel_folder_get_message_flags (folder, changed->uid_changed->pdata [i]); + if (flags & CAMEL_MESSAGE_JUNK_LEARN) { + if (flags & CAMEL_MESSAGE_JUNK) { + if (!junk) + junk = g_ptr_array_new(); + g_ptr_array_add (junk, g_strdup (changed->uid_changed->pdata [i])); + } else { + if (!notjunk) + notjunk = g_ptr_array_new(); + g_ptr_array_add (notjunk, g_strdup (changed->uid_changed->pdata [i])); } + /* reset junk learn flag so that we don't process it again */ + camel_folder_set_message_flags (folder, changed->uid_changed->pdata [i], CAMEL_MESSAGE_JUNK_LEARN, 0); } } - - driver = camel_session_get_filter_driver (session, source, NULL); + d(if (junk || notjunk) printf("** Have '%d' messages for junk filter to learn, launching thread to process them\n", + (junk ? junk->len : 0) + (notjunk ? notjunk->len : 0))); } - + + if ((folder->folder_flags & (CAMEL_FOLDER_FILTER_RECENT|CAMEL_FOLDER_FILTER_JUNK)) + && changed->uid_recent->len > 0) + driver = camel_session_get_filter_driver(session, + (folder->folder_flags & CAMEL_FOLDER_FILTER_RECENT) + ? FILTER_SOURCE_INCOMING : FILTER_SOURCE_JUNKTEST, NULL); + CAMEL_FOLDER_LOCK(folder, change_lock); if (driver) { -- cgit v1.2.3