From: Brett Cannon <bcannon@gmail.com>
Date: Wed, 21 Mar 2007 22:26:20 +0000 (+0000)
Subject: When removing indexing/slicing on exceptions some places were changed
X-Git-Tag: v3.0a1~1102
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ca477b228084b9f8f9c8d5818e49261d708dfdcf;p=python

When removing indexing/slicing on exceptions some places were changed
inappropriately from ``e[0]`` to ``e.message`` instead of ``e.args[0]``.  The
reason it needs to be the last option is the dichotomy of 'message' and 'args':
'message' can be the empty string but args[0] can have a value if more than one
argument was passed.
---

diff --git a/Lib/ConfigParser.py b/Lib/ConfigParser.py
index 9a365c0821..8e644e1695 100644
--- a/Lib/ConfigParser.py
+++ b/Lib/ConfigParser.py
@@ -569,7 +569,7 @@ class ConfigParser(RawConfigParser):
                     value = value % vars
                 except KeyError as e:
                     raise InterpolationMissingOptionError(
-                        option, section, rawval, e.message)
+                        option, section, rawval, e.args[0])
             else:
                 break
         if "%(" in value:
diff --git a/Lib/test/test_file.py b/Lib/test/test_file.py
index a57ab43158..c4bd610d3e 100644
--- a/Lib/test/test_file.py
+++ b/Lib/test/test_file.py
@@ -156,7 +156,7 @@ class OtherFileTests(unittest.TestCase):
         try:
             f = open(TESTFN, bad_mode)
         except ValueError as msg:
-            if msg.message != 0:
+            if msg.args[0] != 0:
                 s = str(msg)
                 if s.find(TESTFN) != -1 or s.find(bad_mode) == -1:
                     self.fail("bad error message for invalid mode: %s" % s)