openstack/swift
Code Issues Proposed changes

bare excepts, as is proper

Browse Source 
Change-Id: Ifd28f6f14a781a67644315690491888161a7250c
This commit is contained in:
John Dickinson 2013-11-26 14:39:30 -08:00
parent 9248c91f6d
commit 71d1805683
2 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
swift/common/memcached.py
View File

@ -150,10 +150,7 @@ class MemcacheConnPool(Pool):
if self.current_size <= self.max_size: if self.current_size <= self.max_size:
try: try:
created = self.create() created = self.create()
# This was really supposed to be "except:" but ran afoul of the except: # noqa
# H201 check, which does not implement the "noqa" exception. Once
# that's fixed, the except here can be changed to "except: # noqa"
except (Exception, BaseException):
self.current_size -= 1 self.current_size -= 1
raise raise
return created return created

6
test/unit/common/test_memcached.py
View File

@ -444,11 +444,7 @@ class TestMemcached(unittest.TestCase):
while not got: while not got:
try: try:
got = pool.get() got = pool.get()
# This was really supposed to be "except:" but ran afoul except: # noqa
# of the H201 check, which does not implement the "noqa"
# exception. Once that's fixed, the except here can be
# changed to "except: # noqa"
except (Exception, BaseException):
pass pass
pool.put(got) pool.put(got)