aboutsummaryrefslogtreecommitdiffstats
path: root/camel
diff options
context:
space:
mode:
authorJeffrey Stedfast <fejj@helixcode.com>2000-06-20 04:54:50 +0800
committerJeffrey Stedfast <fejj@src.gnome.org>2000-06-20 04:54:50 +0800
commitb27eb5f67ae5a2b1bb93e661ac7862520ff30d4b (patch)
treec22e2b73897220118dd44caa45437d546e7792a7 /camel
parentb14b1bfac1a5097036d2ddb8d5fe464515313c78 (diff)
downloadgsoc2013-evolution-b27eb5f67ae5a2b1bb93e661ac7862520ff30d4b.tar.gz
gsoc2013-evolution-b27eb5f67ae5a2b1bb93e661ac7862520ff30d4b.tar.zst
gsoc2013-evolution-b27eb5f67ae5a2b1bb93e661ac7862520ff30d4b.zip
Set summary equal to NULL. (imap_get_summary): Store the summary in the
2000-06-19 Jeffrey Stedfast <fejj@helixcode.com> * providers/imap/camel-imap-folder.c (camel_imap_folder_init): Set summary equal to NULL. (imap_get_summary): Store the summary in the ImapFolder (imap_summary_get_by_uid): If we have a summary cache in the ImapFolder, first check to see if that message info is in the cached summary first, if not fetch it directly from the IMAP server and append it to the summary cache. (imap_get_message_flags): Don't free the message info that we get back from summary_get_by_uid as we don't want to be corrupting our cached summary. svn path=/trunk/; revision=3636
Diffstat (limited to 'camel')
-rw-r--r--camel/ChangeLog13
-rw-r--r--camel/providers/imap/camel-imap-folder.c45
-rw-r--r--camel/providers/imap/camel-imap-folder.h4
3 files changed, 49 insertions, 13 deletions
diff --git a/camel/ChangeLog b/camel/ChangeLog
index df3be4b8ca..5b4d5aabd6 100644
--- a/camel/ChangeLog
+++ b/camel/ChangeLog
@@ -1,3 +1,16 @@
+2000-06-19 Jeffrey Stedfast <fejj@helixcode.com>
+
+ * providers/imap/camel-imap-folder.c (camel_imap_folder_init):
+ Set summary equal to NULL.
+ (imap_get_summary): Store the summary in the ImapFolder
+ (imap_summary_get_by_uid): If we have a summary cache in the
+ ImapFolder, first check to see if that message info is in the
+ cached summary first, if not fetch it directly from the IMAP
+ server and append it to the summary cache.
+ (imap_get_message_flags): Don't free the message info that we
+ get back from summary_get_by_uid as we don't want to be
+ corrupting our cached summary.
+
2000-06-19 Peter Williams <peterw@curious-george.helixcode.com>
* providers/imap/camel-imap-store.c (camel_imap_command{,_extended}): When
diff --git a/camel/providers/imap/camel-imap-folder.c b/camel/providers/imap/camel-imap-folder.c
index c52cc00a09..55d21d9511 100644
--- a/camel/providers/imap/camel-imap-folder.c
+++ b/camel/providers/imap/camel-imap-folder.c
@@ -153,6 +153,7 @@ camel_imap_folder_init (gpointer object, gpointer klass)
folder->has_summary_capability = TRUE;
folder->has_search_capability = FALSE; /* default - we have to query IMAP to know for sure */
+ imap_folder->summary = NULL;
imap_folder->count = -1;
}
@@ -252,6 +253,7 @@ imap_init (CamelFolder *folder, CamelStore *parent_store, CamelFolder *parent_fo
imap_folder->search = NULL;
+ imap_folder->summary = NULL;
/* SELECT the IMAP mail spool */
status = camel_imap_command_extended (CAMEL_IMAP_STORE (folder->parent_store), folder,
@@ -673,10 +675,15 @@ get_header_field (gchar *header, gchar *field)
GPtrArray *
imap_get_summary (CamelFolder *folder, CamelException *ex)
{
+ /* FIXME: we leak mem here if the summary already exists
+ * Q: where do we want to free the pre-existing summary? */
+ CamelImapFolder *imap_folder = CAMEL_IMAP_FOLDER (folder);
GPtrArray *array = NULL;
CamelMessageInfo *info;
int i, num, status;
char *result, *datestr, *p, *q;
+
+ imap_free_summary (folder, imap_folder->summary);
num = imap_get_message_count (folder, ex);
@@ -825,6 +832,8 @@ imap_get_summary (CamelFolder *folder, CamelException *ex)
g_ptr_array_add (array, info);
}
+ imap_folder->summary = array;
+
return array;
}
@@ -846,6 +855,7 @@ imap_free_summary (CamelFolder *folder, GPtrArray *array)
}
g_ptr_array_free (array, TRUE);
+ array = NULL;
return;
}
@@ -854,11 +864,25 @@ imap_free_summary (CamelFolder *folder, GPtrArray *array)
static const CamelMessageInfo *
imap_summary_get_by_uid (CamelFolder *folder, const char *uid)
{
- /* TODO: code this - do a "UID FETCH <uid> BODY.PEEK[HEADER]" and parse */
+ CamelImapFolder *imap_folder = CAMEL_IMAP_FOLDER (folder);
CamelMessageInfo *info = NULL;
char *result, *datestr, *p;
int status;
+ /* lets first check to see if we have the message info cached */
+ if (imap_folder->summary) {
+ int max, i;
+
+ max = imap_folder->summary->len;
+ for (i = 0; i < max; i++) {
+ info = g_ptr_array_index (imap_folder->summary, i);
+ if (!strcmp(info->uid, uid))
+ return info;
+ }
+ }
+
+ /* we don't have a cached copy, so fetch it */
+
status = camel_imap_command_extended (CAMEL_IMAP_STORE (folder->parent_store), folder,
&result, "UID FETCH %s BODY.PEEK[HEADER]", uid);
@@ -919,6 +943,14 @@ imap_summary_get_by_uid (CamelFolder *folder, const char *uid)
info->flags |= CAMEL_MESSAGE_DRAFT;
g_free (result);
+
+ /* since we didn't have it cached, lets add it to our cache */
+ if (imap_folder->summary) {
+ g_ptr_array_add (imap_folder->summary, info);
+ } else {
+ imap_folder->summary = g_ptr_array_new ();
+ g_ptr_array_add (imap_folder->summary, info);
+ }
return info;
}
@@ -966,7 +998,6 @@ static guint32
imap_get_message_flags (CamelFolder *folder, const char *uid, CamelException *ex)
{
CamelMessageInfo *info;
- guint32 flags;
if (!(info = (CamelMessageInfo *)imap_summary_get_by_uid (folder, uid))) {
CamelService *service = CAMEL_SERVICE (folder->parent_store);
@@ -977,15 +1008,7 @@ imap_get_message_flags (CamelFolder *folder, const char *uid, CamelException *ex
return 0;
}
- flags = info->flags;
-
- g_free (info->subject);
- g_free (info->to);
- g_free (info->from);
- g_free (info->uid);
- g_free (info);
-
- return flags;
+ return info->flags;
}
static void
diff --git a/camel/providers/imap/camel-imap-folder.h b/camel/providers/imap/camel-imap-folder.h
index 64483ca147..ea96b2facb 100644
--- a/camel/providers/imap/camel-imap-folder.h
+++ b/camel/providers/imap/camel-imap-folder.h
@@ -45,8 +45,8 @@ extern "C" {
typedef struct {
CamelFolder parent_object;
- CamelFolderSearch *search; /* used to run searches, we just use the real thing (tm) */
-
+ CamelFolderSearch *search; /* used to run searches */
+ GPtrArray *summary;
gint count;
} CamelImapFolder;