diff options
| author | Jeffrey Stedfast <fejj@helixcode.com> | 2000-08-31 12:19:23 +0800 |
|---|---|---|
| committer | Jeffrey Stedfast <fejj@src.gnome.org> | 2000-08-31 12:19:23 +0800 |
| commit | 2f478348f575be5846df82c4cedeac89e6eab31c (patch) | |
| tree | 58da749293aae57e2162669e0e1d50c5dbdb3e07 /camel/providers/imap/camel-imap-folder.c | |
| parent | 3de9db5ae79348a641351f6141b282ce5e9b50a9 (diff) | |
| download | gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.tar gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.tar.gz gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.tar.bz2 gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.tar.lz gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.tar.xz gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.tar.zst gsoc2013-evolution-2f478348f575be5846df82c4cedeac89e6eab31c.zip | |
Don't save any exceptions caused by camel_imap_folder_changed
2000-08-31 Jeffrey Stedfast <fejj@helixcode.com>
* providers/imap/camel-imap-store.c (camel_imap_command_extended):
Don't save any exceptions caused by camel_imap_folder_changed
(camel_imap_fetch_command): Same.
* providers/imap/camel-imap-folder.c (camel_imap_folder_changed):
Using a new way of calculating the first recent message that seems
more accurate. Also added code to make sure we don't accidently
add a duplicate summary.
svn path=/trunk/; revision=5135
Diffstat (limited to 'camel/providers/imap/camel-imap-folder.c')
| -rw-r--r-- | camel/providers/imap/camel-imap-folder.c | 28 |
1 files changed, 24 insertions, 4 deletions
diff --git a/camel/providers/imap/camel-imap-folder.c b/camel/providers/imap/camel-imap-folder.c index 5744a61584..c4c26db300 100644 --- a/camel/providers/imap/camel-imap-folder.c +++ b/camel/providers/imap/camel-imap-folder.c @@ -1482,7 +1482,7 @@ camel_imap_folder_changed (CamelFolder *folder, gint recent, GPtrArray *expunged if (recent > 0) { CamelImapFolder *imap_folder = CAMEL_IMAP_FOLDER (folder); CamelMessageInfo *info; - gint i, j, last; + gint i, j, last, slast; if (!imap_folder->summary) { imap_folder->summary = g_ptr_array_new (); @@ -1490,12 +1490,32 @@ camel_imap_folder_changed (CamelFolder *folder, gint recent, GPtrArray *expunged } last = imap_folder->summary->len + 1; + slast = imap_get_message_count_internal (folder, ex); + fprintf (stderr, "calculated next message is: %d\n", last); + fprintf (stderr, "server says %d mesgs total\n", slast); + slast -= (recent - 1); + fprintf (stderr, "based on total, new guess is: %d\n", slast); - for (i = last, j = 0; j < recent; i++, j++) { + for (i = slast, j = 0; j < recent; i++, j++) { info = imap_get_message_info_internal (folder, i, ex); if (info) { - g_ptr_array_add (imap_folder->summary, info); - g_hash_table_insert (imap_folder->summary_hash, info->uid, info); + if (!imap_get_message_info (folder, info->uid)) { + /* add to our summary */ + g_ptr_array_add (imap_folder->summary, info); + g_hash_table_insert (imap_folder->summary_hash, info->uid, info); + } else { + /* we already have a record of it */ + g_free (info->subject); + g_free (info->from); + g_free (info->to); + g_free (info->cc); + g_free (info->uid); + g_free (info->message_id); + header_references_list_clear (&info->references); + g_free (info); + info = NULL; + d(fprintf (stderr, "we already had message %d!!\n", i)); + } } else { /* our hack failed so now we need to do it the old fashioned way */ /*imap_get_summary_internal (folder, ex);*/ |