From 6481e09084718ac87333e0ba183588142bbde93b Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Thu, 19 Nov 2009 03:11:09 +0000 Subject: [PATCH] Merged revisions 76395 via svnmerge from svn+ssh://pythondev@svn.python.org/python/trunk ........ r76395 | benjamin.peterson | 2009-11-18 21:00:02 -0600 (Wed, 18 Nov 2009) | 1 line #5037 proxy __unicode__ correctly ........ --- Lib/test/test_weakref.py | 11 +++++++++++ Misc/NEWS | 3 +++ Objects/weakrefobject.c | 17 +++++++++++++++++ 3 files changed, 31 insertions(+) diff --git a/Lib/test/test_weakref.py b/Lib/test/test_weakref.py index aeafddab72..ba109aefd2 100644 --- a/Lib/test/test_weakref.py +++ b/Lib/test/test_weakref.py @@ -188,6 +188,17 @@ class ReferencesTestCase(TestBase): self.assertEqual(L3[:5], p3[:5]) self.assertEqual(L3[2:5], p3[2:5]) + def test_proxy_unicode(self): + # See bug 5037 + class C(object): + def __str__(self): + return "string" + def __unicode__(self): + return u"unicode" + instance = C() + self.assertTrue("__unicode__" in dir(weakref.proxy(instance))) + self.assertEqual(unicode(weakref.proxy(instance)), u"unicode") + def test_proxy_index(self): class C: def __index__(self): diff --git a/Misc/NEWS b/Misc/NEWS index c75f369981..4e7d9e3bf1 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -251,6 +251,9 @@ Core and Builtins Library ------- +- Issue #5037: Proxy the __unicode__ special method instead to __unicode__ + instead of __str__. + - Issue #6894: Fixed the issue urllib2 doesn't respect "no_proxy" environment - Issue #6790: Make it possible again to pass an `array.array` to diff --git a/Objects/weakrefobject.c b/Objects/weakrefobject.c index 9cdd02178d..1a998b64bd 100644 --- a/Objects/weakrefobject.c +++ b/Objects/weakrefobject.c @@ -433,6 +433,13 @@ proxy_checkref(PyWeakReference *proxy) return generic(proxy, v, w); \ } +#define WRAP_METHOD(method, special) \ + static PyObject * \ + method(PyObject *proxy) { \ + UNWRAP(proxy); \ + return PyObject_CallMethod(proxy, special, ""); \ + } + /* direct slots */ @@ -593,6 +600,15 @@ proxy_iternext(PyWeakReference *proxy) } +WRAP_METHOD(proxy_unicode, "__unicode__"); + + +static PyMethodDef proxy_methods[] = { + {"__unicode__", (PyCFunction)proxy_unicode, METH_NOARGS}, + {NULL, NULL} +}; + + static PyNumberMethods proxy_as_number = { proxy_add, /*nb_add*/ proxy_sub, /*nb_subtract*/ @@ -684,6 +700,7 @@ _PyWeakref_ProxyType = { 0, /* tp_weaklistoffset */ (getiterfunc)proxy_iter, /* tp_iter */ (iternextfunc)proxy_iternext, /* tp_iternext */ + proxy_methods, /* tp_methods */ }; -- 2.50.1