]> granicus.if.org Git - python/commitdiff
Possibly the end of SF [#460020] bug or feature: unicode() and subclasses.
authorTim Peters <tim.peters@gmail.com>
Tue, 11 Sep 2001 03:07:38 +0000 (03:07 +0000)
committerTim Peters <tim.peters@gmail.com>
Tue, 11 Sep 2001 03:07:38 +0000 (03:07 +0000)
Changed unicode(i) to return a true Unicode object when i is an instance of
a unicode subclass.  Added PyUnicode_CheckExact macro.

Include/unicodeobject.h
Lib/test/test_descr.py
Objects/unicodeobject.c

index dbeb883eb3ceaeb29bd590c88464d73deefd8b3a..d9cda95f937d244bb17d54072cb739c8ee033ec7 100644 (file)
@@ -61,6 +61,7 @@ Copyright (c) Corporation for National Research Initiatives.
 #ifndef Py_USING_UNICODE
 
 #define PyUnicode_Check(op)                 0
+#define PyUnicode_CheckExact(op)            0
 
 #else
 
@@ -373,6 +374,7 @@ typedef struct {
 extern DL_IMPORT(PyTypeObject) PyUnicode_Type;
 
 #define PyUnicode_Check(op) PyObject_TypeCheck(op, &PyUnicode_Type)
+#define PyUnicode_CheckExact(op) ((op)->ob_type == &PyUnicode_Type)
 
 /* Fast access macros */
 #define PyUnicode_GET_SIZE(op) \
index fea32550ca171e6f43ddd4b03c28e186433eb8a5..a2ec4758f667f42322344bfdce625a093066337c 100644 (file)
@@ -1443,7 +1443,7 @@ def inherits():
     verify(str(s).__class__ is str)
 
     s = madstring("\x00" * 5)
-    #XXX verify(str(s) == "\x00" ( 5)
+    #XXX verify(str(s) == "\x00" * 5)
     verify(str(s).__class__ is str)
 
     class madunicode(unicode):
@@ -1460,7 +1460,7 @@ def inherits():
     verify(u.rev().rev() == madunicode(u"ABCDEF"))
     u = madunicode(u"12345")
     verify(unicode(u) == u"12345")
-    #XXX verify(unicode(u).__class__ is unicode)
+    verify(unicode(u).__class__ is unicode)
 
 def all():
     lists()
index 98691fddcd6eeb6c78122e01072d05fe6ff7b6e9..a50b9256c91b411eac12b5f88f44420d89a697b3 100644 (file)
@@ -425,12 +425,20 @@ PyObject *PyUnicode_FromEncodedObject(register PyObject *obj,
     }
     if (PyUnicode_Check(obj)) {
        if (encoding) {
-           PyErr_SetString(PyExc_TypeError,
+            PyErr_SetString(PyExc_TypeError,
                            "decoding Unicode is not supported");
-           return NULL;
+            return NULL;
        }
-       Py_INCREF(obj);
-       v = obj;
+        if (PyUnicode_CheckExact(obj)) {
+           Py_INCREF(obj);
+            v = obj;
+       }
+        else {
+            /* For a subclass of unicode, return a true unicode object
+               with the same string value. */
+            v = PyUnicode_FromUnicode(PyUnicode_AS_UNICODE(obj),
+                                      PyUnicode_GET_SIZE(obj));
+        }
        goto done;
     }
     else if (PyString_Check(obj)) {