]> granicus.if.org Git - python/commitdiff
Reverted 16efa8d27e4c after discussion with Eric.
authorVinay Sajip <vinay_sajip@yahoo.co.uk>
Tue, 15 Apr 2014 12:52:21 +0000 (13:52 +0100)
committerVinay Sajip <vinay_sajip@yahoo.co.uk>
Tue, 15 Apr 2014 12:52:21 +0000 (13:52 +0100)
Lib/test/test_types.py
Misc/NEWS
Python/formatter_unicode.c

index 11d95465a81312a6b3afa47e1fbe803c8f6f6349..ec10752e6a2ae0eac233c1cdd9bf7cd8ed858b54 100644 (file)
@@ -343,8 +343,6 @@ class TypesTests(unittest.TestCase):
         self.assertRaises(ValueError, 3 .__format__, ",n")
         # can't have ',' with 'c'
         self.assertRaises(ValueError, 3 .__format__, ",c")
-        # can't have '#' with 'c'
-        self.assertRaises(ValueError, 3 .__format__, "#c")
 
         # ensure that only int and float type specifiers work
         for format_spec in ([chr(x) for x in range(ord('a'), ord('z')+1)] +
index 26377fe23d136269d7826c031514ae7e074433b1..271898c82e9ca0bf0d911255f84de591040c8c5f 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -34,10 +34,6 @@ Core and Builtins
   replacement fields. It now matches the behavior of str.format() in
   this regard. Patches by Phil Elson and Ramchandra Apte.
 
-- Issue #8931: Make alternate formatting ('#') for type 'c' raise an
-  exception. It had no effect, now trying to specify it is an error.
-  Patch by Torsten Landschoff.
-
 Library
 -------
 
index 056bb76902597e8b959fa53dcd58386102b10c99..e3a814984151b6e85c9afec1c2a090bee0b22513 100644 (file)
@@ -846,13 +846,6 @@ format_long_internal(PyObject *value, const InternalFormatSpec *format,
                             " format specifier 'c'");
             goto done;
         }
-        /* error to request alternate format */
-        if (format->alternate) {
-            PyErr_SetString(PyExc_ValueError,
-                            "Alternate form (#) not allowed with integer"
-                            " format specifier 'c'");
-            goto done;
-        }
 
         /* taken from unicodeobject.c formatchar() */
         /* Integer input truncated to a character */