# HG changeset patch
# User mpm@selenic.com
# Date 1120237267 28800
# Node ID 0ceea19182a9da6ed9f25e13207e58a7c0b55420
# Parent  b9fee419a1bdbb8edae6982d49172344067897ec
transaction: __del__ should do nothing if the journal already exists

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

transaction: __del__ should do nothing if the journal already exists

manifest hash: c85f747ca762b7c446d306cbd7a20f8e566557fb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.0 (GNU/Linux)

iD8DBQFCxXbTywK+sNU5EO8RAvjsAKCcTSzN9OjtoAdaZgtjDs9380ia7ACfcXsx
FcNESqr4nX9b97WTGszJs48=
=C2sL
-----END PGP SIGNATURE-----


diff -r b9fee419a1bd -r 0ceea19182a9 mercurial/transaction.py
--- a/mercurial/transaction.py	Fri Jul 01 08:54:52 2005 -0800
+++ b/mercurial/transaction.py	Fri Jul 01 09:01:07 2005 -0800
@@ -31,9 +31,11 @@
         self.file = open(self.journal, "w")
 
     def __del__(self):
-        if self.entries: self.abort()
-        try: os.unlink(self.journal)
-        except: pass
+        if self.journal:
+            if self.entries: self.abort()
+            self.file.close()
+            try: os.unlink(self.journal)
+            except: pass
 
     def add(self, file, offset):
         if file in self.map: return