Fixing the crashes that I was seeing

--HG--
extra : transplant_source : %3E%B1%BB%DF%3C%22%A6%CC%90Z%E8v%95%962%DD%B2%A1%11%CC

diff --git a/cnotmuch/database.py b/cnotmuch/database.py
index 93c727c67bdc009c34596ef2af5757a7065016e4..d13294334f4ab2050e3418795e0d0b9a69273c73 100644 (file)
--- a/cnotmuch/database.py
+++ b/cnotmuch/database.py
@@ -246,8 +246,7 @@ class Tags(object):
     def next(self):
         nmlib.notmuch_tags_move_to_next(self._tags)
         if not nmlib.notmuch_tags_valid(self._tags):
     def next(self):
         nmlib.notmuch_tags_move_to_next(self._tags)
         if not nmlib.notmuch_tags_valid(self._tags):

-            print("Freeing the Tags now")
-            nmlib.notmuch_tags_destroy (self._tags)
+            self._tags = None

             raise StopIteration
         return Tags._get (self._tags)
 
             raise StopIteration
         return Tags._get (self._tags)
 


@@ -299,8 +298,6 @@ class Messages(object):
 
         nmlib.notmuch_messages_move_to_next(self._msgs)
         if not nmlib.notmuch_messages_valid(self._msgs):
 
         nmlib.notmuch_messages_move_to_next(self._msgs)
         if not nmlib.notmuch_messages_valid(self._msgs):

-            print("Freeing the Messages now")
-            nmlib.notmuch_messages_destroy (self._msgs)

             self._msgs = None
             raise StopIteration
         return Message(Messages._get (self._msgs), self)
             self._msgs = None
             raise StopIteration
         return Message(Messages._get (self._msgs), self)


@@ -339,6 +336,7 @@ class Message(object):
         if msg_p is None:
             NotmuchError(STATUS.NULL_POINTER)
         self._msg = msg_p
         if msg_p is None:
             NotmuchError(STATUS.NULL_POINTER)
         self._msg = msg_p

+        #keep reference to parent, so we keep it alive

         self._parent = parent
         print "Inited Message derived from %s" %(str(parent))
 
         self._parent = parent
         print "Inited Message derived from %s" %(str(parent))