Rev 2559: Remove unnecessary confirmation after acquiring LockDir in http://sourcefrog.net/bzr/dlock

[Martin Pool]

At http://sourcefrog.net/bzr/dlock

------------------------------------------------------------
revno: 2559
revision-id: mbp at sourcefrog.net-20070627081353-f3402usvh4zlcs7q
parent: mbp at sourcefrog.net-20070627081010-k0j4wgcyjjfabbe9
committer: Martin Pool <mbp at sourcefrog.net>
branch nick: dlock
timestamp: Wed 2007-06-27 18:13:53 +1000
message:
  Remove unnecessary confirmation after acquiring LockDir
modified:
  bzrlib/lockdir.py              lockdir.py-20060220222025-98258adf27fbdda3
=== modified file 'bzrlib/lockdir.py'
--- a/bzrlib/lockdir.py	2007-06-27 08:10:10 +0000
+++ b/bzrlib/lockdir.py	2007-06-27 08:13:53 +0000
@@ -223,7 +223,10 @@
 
             self.transport.rename(tmpname, self._held_dir)
             self._lock_held = True
-            self.confirm()
+            # we used to do self.confirm() at this point, but it's really
+            # unnecessary, we have no substantial chance of having it broken
+            # just as it's acquired, and we believe that this lock design is
+            # safe on all platforms.
             # FIXME: we should remove the pending lock if we fail, 
             # https://bugs.launchpad.net/bzr/+bug/109169
         except errors.PermissionDenied: