better fix
authorColin Leroy <colin@colino.net>
Tue, 27 Jul 2004 19:57:52 +0000 (19:57 +0000)
committerColin Leroy <colin@colino.net>
Tue, 27 Jul 2004 19:57:52 +0000 (19:57 +0000)
ChangeLog.claws
PATCHSETS
configure.ac
src/msgcache.c

index 9976260..9b0284f 100644 (file)
@@ -1,3 +1,9 @@
+2004-07-27 [colin]     0.9.12cvs37
+
+       * src/msgcache.c
+               More fixes (i overlook a bit, thanks
+               Christoph for spotting it)
+
 2004-07-27 [colin]     0.9.12cvs36
 
        * src/msgcache.c
index 8a9b0e6..8ba92a8 100644 (file)
--- a/PATCHSETS
+++ b/PATCHSETS
@@ -23,3 +23,4 @@
 ( cvs diff -u -r 1.430 -r 1.431 src/compose.c; ) > 0.9.12cvs34.patchset
 ( cvs diff -u -r 1.65 -r 1.66 src/pop.c; ) > 0.9.12cvs35.patchset
 ( cvs diff -u -r 1.24 -r 1.25 src/msgcache.c; ) > 0.9.12cvs36.patchset
+( cvs diff -u -r 1.25 -r 1.26 src/msgcache.c; ) > 0.9.12cvs37.patchset
index fc31380..d08db22 100644 (file)
@@ -11,7 +11,7 @@ MINOR_VERSION=9
 MICRO_VERSION=12
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=36
+EXTRA_VERSION=37
 EXTRA_RELEASE=
 
 if test \( $EXTRA_VERSION -eq 0 \) -o \( "x$EXTRA_RELEASE" != "x" \); then
index f9fb466..61bd755 100644 (file)
@@ -116,14 +116,16 @@ void msgcache_update_msg(MsgCache *cache, MsgInfo *msginfo)
        g_return_if_fail(msginfo != NULL);
 
        oldmsginfo = g_hash_table_lookup(cache->msgnum_table, &msginfo->msgnum);
+
        if(oldmsginfo && oldmsginfo->msgid) {
                g_hash_table_remove(cache->msgid_table, oldmsginfo->msgid);
+       }
+
+       if (oldmsginfo) {
                g_hash_table_remove(cache->msgnum_table, &oldmsginfo->msgnum);
-       } 
-       if (oldmsginfo)
                procmsg_msginfo_free(oldmsginfo);
-
-       cache->memusage -= procmsg_msginfo_memusage(oldmsginfo);
+               cache->memusage -= procmsg_msginfo_memusage(oldmsginfo);
+       }
 
        newmsginfo = procmsg_msginfo_new_ref(msginfo);
        g_hash_table_insert(cache->msgnum_table, &newmsginfo->msgnum, newmsginfo);