From cbdb05978b8be7764ed3efd9f773b570df58b7bb Mon Sep 17 00:00:00 2001 From: Kovid Goyal Date: Tue, 31 Jul 2012 17:41:44 +0530 Subject: [PATCH] Use context manager instead of acquire()/release() --- src/calibre/devices/smart_device_app/driver.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/calibre/devices/smart_device_app/driver.py b/src/calibre/devices/smart_device_app/driver.py index c63c04b7a6..8f59d310c8 100644 --- a/src/calibre/devices/smart_device_app/driver.py +++ b/src/calibre/devices/smart_device_app/driver.py @@ -36,13 +36,9 @@ def synchronous(tlockname): def _synched(func): @wraps(func) - def _synchronizer(self,*args, **kwargs): - tlock = self.__getattribute__( tlockname) - tlock.acquire() - try: + def _synchronizer(self, *args, **kwargs): + with self.__getattribute__(tlockname): return func(self, *args, **kwargs) - finally: - tlock.release() return _synchronizer return _synched