/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/counted_lock.py

  • Committer: Vincent Ladeuil
  • Date: 2007-06-24 15:16:40 UTC
  • mto: (2547.2.1 jam-integration)
  • mto: This revision was merged to the branch mainline in revision 2550.
  • Revision ID: v.ladeuil+lp@free.fr-20070624151640-5tg62jb5g9tqvybd
Fix 121889 by working around urllib2 bug.

* tests/HTTPTestUtil.py:
(DigestAuthRequestHandler.send_header_auth_reqed): python-2.4.1
fail to decode the header without the quotes. The RFC do not
require them, python >= 2.4.4 handles them gracefully. Not a big
deal.

* tests/test_http.py:
(TestAuth.setUp): Add a comment in hope I will not running around
shouting: "Who takes my traces ? Gimme my traces !" when running
the only tests who capture their own traces without showing them.

* transport/http/_urllib2_wrappers.py (BasicAuthHandler,
DigestAuthHandler): Not directly related to the bug, bug good to
fix anyway, the digest auth should be preferred to the basic
one. To do so, the digest handler should be tried before the basic
one.

* builtins.py:
(cmd_selftest.run): Fix typo. Note to reviewers: No, it's not
related to the bug. No there are no tests for that. No I don't
intend to write some :) But I'll understand if you veto that
because you want to take care of it :D

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2007 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Counted lock class"""
 
18
 
 
19
 
 
20
from bzrlib.errors import (
 
21
    LockError,
 
22
    ReadOnlyError,
 
23
    )
 
24
 
 
25
 
 
26
# TODO: Pass through lock tokens on lock_write and read, and return them...
 
27
#
 
28
# TODO: Allow upgrading read locks to write?  Conceptually difficult.
 
29
 
 
30
 
 
31
class CountedLock(object):
 
32
    """Decorator around a lock that makes it reentrant.
 
33
 
 
34
    This can be used with any object that provides a basic Lock interface,
 
35
    including LockDirs and OS file locks.
 
36
    """
 
37
 
 
38
    def __init__(self, real_lock):
 
39
        self._real_lock = real_lock
 
40
        self._lock_mode = None
 
41
        self._lock_count = 0
 
42
 
 
43
    def break_lock(self):
 
44
        self._real_lock.break_lock()
 
45
        self._lock_mode = None
 
46
        self._lock_count = 0
 
47
 
 
48
    def is_locked(self):
 
49
        return self._lock_mode is not None
 
50
 
 
51
    def lock_read(self):
 
52
        """Acquire the lock in read mode.
 
53
 
 
54
        If the lock is already held in either read or write mode this
 
55
        increments the count and succeeds.  If the lock is not already held,
 
56
        it is taken in read mode.
 
57
        """
 
58
        if self._lock_mode:
 
59
            assert self._lock_mode in ('r', 'w'), \
 
60
                   "invalid lock mode %r" % self._lock_mode
 
61
            self._lock_count += 1
 
62
        else:
 
63
            assert self._lock_count == 0
 
64
            self._real_lock.lock_read()
 
65
            self._lock_count = 1
 
66
            self._lock_mode = 'r'
 
67
 
 
68
    def lock_write(self):
 
69
        """Acquire the lock in write mode.
 
70
 
 
71
        If the lock was originally acquired in read mode this will fail.
 
72
        """
 
73
        if self._lock_count == 0:
 
74
            assert self._lock_mode is None
 
75
            self._real_lock.lock_write()
 
76
            self._lock_mode = 'w'
 
77
        elif self._lock_mode != 'w':
 
78
            raise ReadOnlyError(self)
 
79
        self._lock_count += 1
 
80
 
 
81
    def unlock(self):
 
82
        if self._lock_count == 0:
 
83
            raise LockError("%s not locked" % (self,))
 
84
        elif self._lock_count == 1:
 
85
            self._real_lock.unlock()
 
86
            self._lock_mode = None
 
87
        self._lock_count -= 1