From d184f178b8ede3b173789195ba22a243da08fc42 Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Thu, 18 Jan 2001 02:49:29 +0000 Subject: Add some g_warnings() to help debug later if I ever get the segfaults I 2001-01-17 Jeffrey Stedfast * providers/imap/camel-imap-summary.c (message_info_load): Add some g_warnings() to help debug later if I ever get the segfaults I was getting earlier. Can't seem to reproduce them after my previous "fix". * camel-folder-summary.c (camel_folder_summary_load): Undid my change here, NotZed said it was wrong and that if it gets a NULL mi then it should abort and not continue. svn path=/trunk/; revision=7609 --- camel/ChangeLog | 11 +++++++++++ camel/camel-folder-summary.c | 3 --- camel/providers/imap/camel-imap-summary.c | 6 +++++- 3 files changed, 16 insertions(+), 4 deletions(-) diff --git a/camel/ChangeLog b/camel/ChangeLog index e485d8f443..49871b60e6 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,3 +1,14 @@ +2001-01-17 Jeffrey Stedfast + + * providers/imap/camel-imap-summary.c (message_info_load): Add + some g_warnings() to help debug later if I ever get the segfaults + I was getting earlier. Can't seem to reproduce them after my + previous "fix". + + * camel-folder-summary.c (camel_folder_summary_load): Undid my + change here, NotZed said it was wrong and that if it gets a NULL + mi then it should abort and not continue. + 2001-01-17 Jeffrey Stedfast * camel-folder-summary.c (camel_folder_summary_load): Make sure diff --git a/camel/camel-folder-summary.c b/camel/camel-folder-summary.c index 8142029a15..147c8e452e 100644 --- a/camel/camel-folder-summary.c +++ b/camel/camel-folder-summary.c @@ -537,9 +537,6 @@ camel_folder_summary_load(CamelFolderSummary *s) for (i=0;isaved_count;i++) { mi = ((CamelFolderSummaryClass *)(CAMEL_OBJECT_GET_CLASS(s)))->message_info_load(s, in); - if (!mi) - continue; - if (s->build_content) { mi->content = perform_content_info_load(s, in); } diff --git a/camel/providers/imap/camel-imap-summary.c b/camel/providers/imap/camel-imap-summary.c index dd77673cad..74a024f760 100644 --- a/camel/providers/imap/camel-imap-summary.c +++ b/camel/providers/imap/camel-imap-summary.c @@ -169,11 +169,15 @@ message_info_load (CamelFolderSummary *s, FILE *in) CamelImapMessageInfo *iinfo; info = camel_imap_summary_parent->message_info_load (s, in); - if (!info) + if (!info) { + g_warning ("eek! encountered a NULL message info!"); return NULL; + } iinfo = (CamelImapMessageInfo *)info; if (camel_folder_summary_decode_uint32 (in, &iinfo->server_flags) == -1) { + /* wouldn't it just be better to default to certain server flags here? */ + g_warning ("eek! problems decoding server flags!"); camel_folder_summary_info_free (s, info); return NULL; } -- cgit v1.2.3