diff options
author | Antoine Pitrou <solipsis@pitrou.net> | 2011-10-11 22:45:48 +0200 |
---|---|---|
committer | Antoine Pitrou <solipsis@pitrou.net> | 2011-10-11 22:45:48 +0200 |
commit | eb10df871e099882f59eafbc8bdc28c9ebcd7e6d (patch) | |
tree | 46cea1e5d1c5054d7b352752ed50bce36552ec2d /Objects | |
parent | 961927671078881e97d4d3d369755f94ed4e5dc3 (diff) | |
download | cpython-eb10df871e099882f59eafbc8bdc28c9ebcd7e6d.tar.gz |
Use _PyUnicode_CONVERT_BYTES() where applicable.
Diffstat (limited to 'Objects')
-rw-r--r-- | Objects/unicodeobject.c | 29 |
1 files changed, 17 insertions, 12 deletions
diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c index cccab539f8..b2b492a9ec 100644 --- a/Objects/unicodeobject.c +++ b/Objects/unicodeobject.c @@ -1757,14 +1757,14 @@ _PyUnicode_FromUCS4(const Py_UCS4 *u, Py_ssize_t size) res = PyUnicode_New(size, max_char); if (!res) return NULL; - if (max_char >= 0x10000) + if (max_char < 256) + _PyUnicode_CONVERT_BYTES(Py_UCS4, Py_UCS1, u, u + size, + PyUnicode_1BYTE_DATA(res)); + else if (max_char < 0x10000) + _PyUnicode_CONVERT_BYTES(Py_UCS4, Py_UCS2, u, u + size, + PyUnicode_2BYTE_DATA(res)); + else memcpy(PyUnicode_4BYTE_DATA(res), u, sizeof(Py_UCS4)*size); - else { - int kind = PyUnicode_KIND(res); - void *data = PyUnicode_DATA(res); - for (i = 0; i < size; i++) - PyUnicode_WRITE(kind, data, i, u[i]); - } assert(_PyUnicode_CheckConsistency(res, 1)); return res; } @@ -1978,13 +1978,18 @@ as_ucs4(PyObject *string, Py_UCS4 *target, Py_ssize_t targetsize, return NULL; } } - if (kind != PyUnicode_4BYTE_KIND) { - Py_ssize_t i; - for (i = 0; i < len; i++) - target[i] = PyUnicode_READ(kind, data, i); + if (kind == PyUnicode_1BYTE_KIND) { + Py_UCS1 *start = (Py_UCS1 *) data; + _PyUnicode_CONVERT_BYTES(Py_UCS1, Py_UCS4, start, start + len, target); } - else + else if (kind == PyUnicode_2BYTE_KIND) { + Py_UCS2 *start = (Py_UCS2 *) data; + _PyUnicode_CONVERT_BYTES(Py_UCS2, Py_UCS4, start, start + len, target); + } + else { + assert(kind == PyUnicode_4BYTE_KIND); Py_MEMCPY(target, data, len * sizeof(Py_UCS4)); + } if (copy_null) target[len] = 0; return target; |