[PATCH v2] refactoring: simplify the semaphorewait logic

[Sebastian Spaeth]

On Wed, 11 May 2011 22:10:56 +0200, Nicolas Sebrecht <nicolas.s-dev at laposte.net> wrote:
> The semaphorewait()/waitforthread() logic is usefull for IMAP starting
> connections. We actually use it in imapserver only.
> -        threadutil.semaphorewait(self.semaphore)
> +        self.semaphore.acquire()
> +        self.semaphore.release()

Reviewed-by: Sebastian Spaeth <Sebastian at SSpaeth.de>

The patch is ok, but in the longer term, I would love to rename the
variable self.semaphore to something more useful. It is a semaphore for
WHAT? That is a very useless name ...

Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/offlineimap-project/attachments/20110511/16dcd4fd/attachment-0001.pgp>