From: Benjamin Peterson <benjamin@python.org>
Date: Wed, 15 Apr 2009 21:40:50 +0000 (+0000)
Subject: Merged revisions 71628 via svnmerge from
X-Git-Tag: 3.0~224
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9b5a8ea3ed5ac04bc8c25face43fcde738499c3b;p=thirdparty%2FPython%2Fcpython.git

Merged revisions 71628 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k

................
  r71628 | benjamin.peterson | 2009-04-15 16:34:27 -0500 (Wed, 15 Apr 2009) | 11 lines

  Merged revisions 71627 via svnmerge from
  svn+ssh://pythondev@svn.python.org/python/trunk

  ........
    r71627 | benjamin.peterson | 2009-04-15 16:26:36 -0500 (Wed, 15 Apr 2009) | 4 lines

    call __float__ on str subclasses #5759

    tests by R. David Murray
  ........
................
---

diff --git a/Lib/test/test_float.py b/Lib/test/test_float.py
index 2d3b3575ccbb..123a69d404d3 100644
--- a/Lib/test/test_float.py
+++ b/Lib/test/test_float.py
@@ -78,11 +78,18 @@ class GeneralFloatCases(unittest.TestCase):
             def __float__(self):
                 return 42
 
+        # Issue 5759: __float__ not called on str subclasses (though it is on
+        # unicode subclasses).
+        class FooStr(str):
+            def __float__(self):
+                return float(str(self)) + 1
+
         self.assertAlmostEqual(float(Foo0()), 42.)
         self.assertAlmostEqual(float(Foo1()), 42.)
         self.assertAlmostEqual(float(Foo2()), 42.)
         self.assertAlmostEqual(float(Foo3(21)), 42.)
         self.assertRaises(TypeError, float, Foo4(42))
+        self.assertAlmostEqual(float(FooStr('8')), 9.)
 
     def test_floatasratio(self):
         for f, ratio in [
diff --git a/Misc/NEWS b/Misc/NEWS
index 8d65ae4c9a93..356d36aa7646 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -16,6 +16,8 @@ Core and Builtins
   imp.find_module() were converted to UTF-8 while the path is
   converted to the default filesystem encoding, causing nonsense.
 
+- Issue #5759: float() didn't call __float__ on str subclasses.
+
 - Issue #5392: when a very low recursion limit was set, the interpreter would
   abort with a fatal error after the recursion limit was hit twice.
 
diff --git a/Objects/floatobject.c b/Objects/floatobject.c
index e77b2dc855ef..2ef4d1a865a1 100644
--- a/Objects/floatobject.c
+++ b/Objects/floatobject.c
@@ -1533,7 +1533,9 @@ float_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
 		return float_subtype_new(type, args, kwds); /* Wimp out */
 	if (!PyArg_ParseTupleAndKeywords(args, kwds, "|O:float", kwlist, &x))
 		return NULL;
-	if (PyUnicode_Check(x))
+	/* If it's a string, but not a string subclass, use
+	   PyFloat_FromString. */
+	if (PyUnicode_CheckExact(x))
 		return PyFloat_FromString(x);
 	return PyNumber_Float(x);
 }