[Lldb-commits] [lldb] r251968 - Python 3 - modernize exception catching syntax.
Zachary Turner via lldb-commits
lldb-commits at lists.llvm.org
Tue Nov 3 11:49:05 PST 2015
Author: zturner
Date: Tue Nov 3 13:49:05 2015
New Revision: 251968
URL: http://llvm.org/viewvc/llvm-project?rev=251968&view=rev
Log:
Python 3 - modernize exception catching syntax.
Old-style syntax: `except Exception, e:`
New-style syntax: `except Exception as e:`
These two statements are identical, except that the former has
been deprecated for may versions, and was removed in Python 3.
This converts everything to use the new syntax (which also works
in Python 2). I had to convert unittest2 as well. What we really
need to do is just delete unittest2, and use unittest instead since
it is a standard module that ships with every Python distribution.
But this is the path of least resistance for now, although at
some point we will really need to do it.
Modified:
lldb/trunk/examples/synthetic/unordered_multi.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/case.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/loader.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/main.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/suite.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/test/support.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_assertions.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_case.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_loader.py
lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_unittest2_with.py
Modified: lldb/trunk/examples/synthetic/unordered_multi.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/examples/synthetic/unordered_multi.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/examples/synthetic/unordered_multi.py (original)
+++ lldb/trunk/examples/synthetic/unordered_multi.py Tue Nov 3 13:49:05 2015
@@ -52,7 +52,7 @@ class libcxx_hash_table_SynthProvider:
self.next_element = self.begin_ptr
else:
self.next_element = None
- except Exception, e:
+ except Exception as e:
logger >> "Caught exception: %r" % e
pass
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/case.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/case.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/case.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/case.py Tue Nov 3 13:49:05 2015
@@ -348,7 +348,7 @@ class TestCase(unittest.TestCase):
success = False
try:
self.setUp()
- except SkipTest, e:
+ except SkipTest as e:
self._addSkip(result, str(e))
except Exception:
result.addError(self, sys.exc_info())
@@ -386,7 +386,7 @@ class TestCase(unittest.TestCase):
testMethod()
except self.failureException:
result.addFailure(self, sys.exc_info())
- except _ExpectedFailure, e:
+ except _ExpectedFailure as e:
addExpectedFailure = getattr(result, 'addExpectedFailure', None)
if addExpectedFailure is not None:
addExpectedFailure(self, e.exc_info, e.bugnumber)
@@ -394,7 +394,7 @@ class TestCase(unittest.TestCase):
warnings.warn("Use of a TestResult without an addExpectedFailure method is deprecated",
DeprecationWarning)
result.addSuccess(self)
- except _UnexpectedSuccess, x:
+ except _UnexpectedSuccess as x:
addUnexpectedSuccess = getattr(result, 'addUnexpectedSuccess', None)
if addUnexpectedSuccess is not None:
addUnexpectedSuccess(self, x.bugnumber)
@@ -402,7 +402,7 @@ class TestCase(unittest.TestCase):
warnings.warn("Use of a TestResult without an addUnexpectedSuccess method is deprecated",
DeprecationWarning)
result.addFailure(self, sys.exc_info())
- except SkipTest, e:
+ except SkipTest as e:
self._addSkip(result, str(e))
except Exception:
result.addError(self, sys.exc_info())
@@ -807,16 +807,16 @@ class TestCase(unittest.TestCase):
"""
try:
difference1 = set1.difference(set2)
- except TypeError, e:
+ except TypeError as e:
self.fail('invalid type when attempting set difference: %s' % e)
- except AttributeError, e:
+ except AttributeError as e:
self.fail('first argument does not support set difference: %s' % e)
try:
difference2 = set2.difference(set1)
- except TypeError, e:
+ except TypeError as e:
self.fail('invalid type when attempting set difference: %s' % e)
- except AttributeError, e:
+ except AttributeError as e:
self.fail('second argument does not support set difference: %s' % e)
if not (difference1 or difference2):
@@ -1017,7 +1017,7 @@ class TestCase(unittest.TestCase):
return _AssertRaisesContext(expected_exception, self, expected_regexp)
try:
callable_obj(*args, **kwargs)
- except expected_exception, exc_value:
+ except expected_exception as exc_value:
if isinstance(expected_regexp, basestring):
expected_regexp = re.compile(expected_regexp)
if not expected_regexp.search(str(exc_value)):
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/loader.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/loader.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/loader.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/loader.py Tue Nov 3 13:49:05 2015
@@ -89,7 +89,7 @@ class TestLoader(unittest.TestLoader):
if use_load_tests and load_tests is not None:
try:
return load_tests(self, tests, None)
- except Exception, e:
+ except Exception as e:
return _make_failed_load_tests(module.__name__, e,
self.suiteClass)
return tests
@@ -295,7 +295,7 @@ class TestLoader(unittest.TestLoader):
else:
try:
yield load_tests(self, tests, pattern)
- except Exception, e:
+ except Exception as e:
yield _make_failed_load_tests(package.__name__, e,
self.suiteClass)
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/main.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/main.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/main.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/main.py Tue Nov 3 13:49:05 2015
@@ -150,7 +150,7 @@ class TestProgram(object):
else:
self.testNames = (self.defaultTest,)
self.createTests()
- except getopt.error, msg:
+ except getopt.error as msg:
self.usageExit(msg)
def createTests(self):
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/suite.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/suite.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/suite.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/suite.py Tue Nov 3 13:49:05 2015
@@ -138,7 +138,7 @@ class TestSuite(BaseTestSuite):
if setUpClass is not None:
try:
setUpClass()
- except Exception, e:
+ except Exception as e:
if isinstance(result, _DebugResult):
raise
currentClass._classSetupFailed = True
@@ -172,7 +172,7 @@ class TestSuite(BaseTestSuite):
if setUpModule is not None:
try:
setUpModule()
- except Exception, e:
+ except Exception as e:
if isinstance(result, _DebugResult):
raise
result._moduleSetUpFailed = True
@@ -203,7 +203,7 @@ class TestSuite(BaseTestSuite):
if tearDownModule is not None:
try:
tearDownModule()
- except Exception, e:
+ except Exception as e:
if isinstance(result, _DebugResult):
raise
errorName = 'tearDownModule (%s)' % previousModule
@@ -225,7 +225,7 @@ class TestSuite(BaseTestSuite):
if tearDownClass is not None:
try:
tearDownClass()
- except Exception, e:
+ except Exception as e:
if isinstance(result, _DebugResult):
raise
className = util.strclass(previousClass)
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/test/support.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/test/support.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/test/support.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/test/support.py Tue Nov 3 13:49:05 2015
@@ -109,7 +109,7 @@ class HashingMixin(object):
self.fail("%r and %r do not hash equal" % (obj_1, obj_2))
except KeyboardInterrupt:
raise
- except Exception, e:
+ except Exception as e:
self.fail("Problem hashing %r and %r: %s" % (obj_1, obj_2, e))
for obj_1, obj_2 in self.ne_pairs:
@@ -119,7 +119,7 @@ class HashingMixin(object):
(obj_1, obj_2))
except KeyboardInterrupt:
raise
- except Exception, e:
+ except Exception as e:
self.fail("Problem hashing %s and %s: %s" % (obj_1, obj_2, e))
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_assertions.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_assertions.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_assertions.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_assertions.py Tue Nov 3 13:49:05 2015
@@ -62,7 +62,7 @@ class Test_Assertions(unittest2.TestCase
self.assertNotRegexpMatches('Ala ma kota', r'r+')
try:
self.assertNotRegexpMatches('Ala ma kota', r'k.t', 'Message')
- except self.failureException, e:
+ except self.failureException as e:
self.assertIn("'kot'", e.args[0])
self.assertIn('Message', e.args[0])
else:
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_case.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_case.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_case.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_case.py Tue Nov 3 13:49:05 2015
@@ -893,7 +893,7 @@ test case
try:
self.assertMultiLineEqual(type_changer(sample_text),
type_changer(revised_sample_text))
- except self.failureException, e:
+ except self.failureException as e:
# need to remove the first line of the error message
error = str(e).encode('utf8').split('\n', 1)[1]
@@ -913,7 +913,7 @@ test case
self.maxDiff = len(diff)//2
try:
self.assertSequenceEqual(seq1, seq2)
- except self.failureException, e:
+ except self.failureException as e:
msg = e.args[0]
else:
self.fail('assertSequenceEqual did not fail.')
@@ -923,7 +923,7 @@ test case
self.maxDiff = len(diff) * 2
try:
self.assertSequenceEqual(seq1, seq2)
- except self.failureException, e:
+ except self.failureException as e:
msg = e.args[0]
else:
self.fail('assertSequenceEqual did not fail.')
@@ -933,7 +933,7 @@ test case
self.maxDiff = None
try:
self.assertSequenceEqual(seq1, seq2)
- except self.failureException, e:
+ except self.failureException as e:
msg = e.args[0]
else:
self.fail('assertSequenceEqual did not fail.')
@@ -961,7 +961,7 @@ test case
test._truncateMessage = truncate
try:
test.assertDictEqual({}, {1: 0})
- except self.failureException, e:
+ except self.failureException as e:
self.assertEqual(str(e), 'foo')
else:
self.fail('assertDictEqual did not fail')
@@ -973,7 +973,7 @@ test case
test._truncateMessage = truncate
try:
test.assertMultiLineEqual('foo', 'bar')
- except self.failureException, e:
+ except self.failureException as e:
self.assertEqual(str(e), 'foo')
else:
self.fail('assertMultiLineEqual did not fail')
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_loader.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_loader.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_loader.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_loader.py Tue Nov 3 13:49:05 2015
@@ -205,7 +205,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromName('')
- except ValueError, e:
+ except ValueError as e:
self.assertEqual(str(e), "Empty module name")
else:
self.fail("TestLoader.loadTestsFromName failed to raise ValueError")
@@ -238,7 +238,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromName('sdasfasfasdf')
- except ImportError, e:
+ except ImportError as e:
self.assertEqual(str(e), "No module named sdasfasfasdf")
else:
self.fail("TestLoader.loadTestsFromName failed to raise ImportError")
@@ -254,7 +254,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromName('unittest2.sdasfasfasdf')
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "'module' object has no attribute 'sdasfasfasdf'")
else:
self.fail("TestLoader.loadTestsFromName failed to raise AttributeError")
@@ -271,7 +271,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromName('sdasfasfasdf', unittest2)
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "'module' object has no attribute 'sdasfasfasdf'")
else:
self.fail("TestLoader.loadTestsFromName failed to raise AttributeError")
@@ -425,7 +425,7 @@ class Test_TestLoader(unittest2.TestCase
loader = unittest2.TestLoader()
try:
loader.loadTestsFromName('testcase_1.testfoo', m)
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "type object 'MyTestCase' has no attribute 'testfoo'")
else:
self.fail("Failed to raise AttributeError")
@@ -583,7 +583,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromNames([''])
- except ValueError, e:
+ except ValueError as e:
self.assertEqual(str(e), "Empty module name")
else:
self.fail("TestLoader.loadTestsFromNames failed to raise ValueError")
@@ -618,7 +618,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromNames(['sdasfasfasdf'])
- except ImportError, e:
+ except ImportError as e:
self.assertEqual(str(e), "No module named sdasfasfasdf")
else:
self.fail("TestLoader.loadTestsFromNames failed to raise ImportError")
@@ -634,7 +634,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromNames(['unittest2.sdasfasfasdf', 'unittest2'])
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "'module' object has no attribute 'sdasfasfasdf'")
else:
self.fail("TestLoader.loadTestsFromNames failed to raise AttributeError")
@@ -653,7 +653,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromNames(['sdasfasfasdf'], unittest2)
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "'module' object has no attribute 'sdasfasfasdf'")
else:
self.fail("TestLoader.loadTestsFromName failed to raise AttributeError")
@@ -672,7 +672,7 @@ class Test_TestLoader(unittest2.TestCase
try:
loader.loadTestsFromNames(['TestCase', 'sdasfasfasdf'], unittest2)
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "'module' object has no attribute 'sdasfasfasdf'")
else:
self.fail("TestLoader.loadTestsFromName failed to raise AttributeError")
@@ -821,7 +821,7 @@ class Test_TestLoader(unittest2.TestCase
loader = unittest2.TestLoader()
try:
loader.loadTestsFromNames(['testcase_1.testfoo'], m)
- except AttributeError, e:
+ except AttributeError as e:
self.assertEqual(str(e), "type object 'MyTestCase' has no attribute 'testfoo'")
else:
self.fail("Failed to raise AttributeError")
Modified: lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_unittest2_with.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_unittest2_with.py?rev=251968&r1=251967&r2=251968&view=diff
==============================================================================
--- lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_unittest2_with.py (original)
+++ lldb/trunk/third_party/Python/module/unittest2/unittest2/test/test_unittest2_with.py Tue Nov 3 13:49:05 2015
@@ -35,7 +35,7 @@ class TestWith(unittest2.TestCase):
self.assertRaises(KeyError, _raise, KeyError("key"))
try:
self.assertRaises(KeyError, lambda: None)
- except self.failureException, e:
+ except self.failureException as e:
self.assertIn("KeyError not raised", e.args)
else:
self.fail("assertRaises() didn't fail")
@@ -48,7 +48,7 @@ class TestWith(unittest2.TestCase):
with self.assertRaises(KeyError) as cm:
try:
raise KeyError
- except Exception, e:
+ except Exception as e:
raise
self.assertIs(cm.exception, e)
@@ -57,7 +57,7 @@ class TestWith(unittest2.TestCase):
try:
with self.assertRaises(KeyError):
pass
- except self.failureException, e:
+ except self.failureException as e:
self.assertIn("KeyError not raised", e.args)
else:
self.fail("assertRaises() didn't fail")
More information about the lldb-commits
mailing list