Rev 2331: Raise LockContetion rather than LockError in http://bzr.arbash-meinel.com/branches/bzr/0.15-dev/locking_errors

[John Arbash Meinel]

At http://bzr.arbash-meinel.com/branches/bzr/0.15-dev/locking_errors

------------------------------------------------------------
revno: 2331
revision-id: john at arbash-meinel.com-20070328201125-l1y0uyia91q56pmf
parent: pqm at pqm.ubuntu.com-20070326124437-6f0ae3c58fa4b765
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: locking_errors
timestamp: Wed 2007-03-28 15:11:25 -0500
message:
  Raise LockContetion rather than LockError
modified:
  bzrlib/lock.py                 lock.py-20050527050856-ec090bb51bc03349
-------------- next part --------------
=== modified file 'bzrlib/lock.py'
--- a/bzrlib/lock.py	2007-03-19 22:11:28 +0000
+++ b/bzrlib/lock.py	2007-03-28 20:11:25 +0000
@@ -143,7 +143,7 @@
                     self.unlock()
                 # we should be more precise about whats a locking
                 # error and whats a random-other error
-                raise errors.LockError(e)
+                raise errors.LockContention(e)
 
         def unlock(self):
             _fcntl_WriteLock._open_locks.remove(self.filename)
@@ -167,7 +167,7 @@
             except IOError, e:
                 # we should be more precise about whats a locking
                 # error and whats a random-other error
-                raise errors.LockError(e)
+                raise errors.LockContention(e)
 
         def unlock(self):
             count = _fcntl_ReadLock._open_locks[self.filename]
@@ -231,7 +231,7 @@
                 fcntl.lockf(new_f, fcntl.LOCK_SH | fcntl.LOCK_NB)
             except IOError, e:
                 # TODO: Raise a more specific error based on the type of error
-                raise errors.LockError(e)
+                raise errors.LockContention(e)
             _fcntl_WriteLock._open_locks.add(self.filename)
 
             self.f = new_f
@@ -276,7 +276,7 @@
                 raise
             except Exception, e:
                 self._clear_f()
-                raise errors.LockError(e)
+                raise errors.LockContention(e)
 
         def unlock(self):
             overlapped = pywintypes.OVERLAPPED()
@@ -284,7 +284,7 @@
                 win32file.UnlockFileEx(self.hfile, 0, 0x7fff0000, overlapped)
                 self._clear_f()
             except Exception, e:
-                raise errors.LockError(e)
+                raise errors.LockContention(e)
 
 
     class _w32c_ReadLock(_w32c_FileLock):
@@ -393,8 +393,8 @@
                 last_err = _GetLastError()
                 if last_err in (ERROR_LOCK_VIOLATION,):
                     raise errors.LockContention(filename)
-                raise errors.LockError('Unknown locking error: %s'
-                                       % (last_err,))
+                raise errors.LockContention('Unknown locking error: %s'
+                                            % (last_err,))
 
         def unlock(self):
             overlapped = OVERLAPPED()
@@ -408,8 +408,8 @@
             if result == 0:
                 self._clear_f()
                 last_err = _GetLastError()
-                raise errors.LockError('Unknown unlocking error: %s'
-                                       % (last_err,))
+                raise errors.LockContention('Unknown unlocking error: %s'
+                                            % (last_err,))
 
 
     class _ctypes_ReadLock(_ctypes_FileLock):