From 7270b7f1aae31ba04885aa5e400e6542a20c789e Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Sun, 17 Aug 2014 21:14:46 +0200 Subject: _pickle: Optimize raw_unicode_escape(), use directly a bytes object, don't use a temporary bytearray object. --- Modules/_pickle.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/Modules/_pickle.c b/Modules/_pickle.c index 665a324..1c15190 100644 --- a/Modules/_pickle.c +++ b/Modules/_pickle.c @@ -2050,7 +2050,7 @@ save_bytes(PicklerObject *self, PyObject *obj) static PyObject * raw_unicode_escape(PyObject *obj) { - PyObject *repr, *result; + PyObject *repr; char *p; Py_ssize_t i, size, expandsize; void *data; @@ -2069,13 +2069,14 @@ raw_unicode_escape(PyObject *obj) if (size > PY_SSIZE_T_MAX / expandsize) return PyErr_NoMemory(); - repr = PyByteArray_FromStringAndSize(NULL, expandsize * size); + repr = PyBytes_FromStringAndSize(NULL, expandsize * size); if (repr == NULL) return NULL; if (size == 0) - goto done; + return repr; + assert(Py_REFCNT(repr) == 1); - p = PyByteArray_AS_STRING(repr); + p = PyBytes_AS_STRING(repr); for (i=0; i < size; i++) { Py_UCS4 ch = PyUnicode_READ(kind, data, i); /* Map 32-bit characters to '\Uxxxxxxxx' */ @@ -2104,12 +2105,10 @@ raw_unicode_escape(PyObject *obj) else *p++ = (char) ch; } - size = p - PyByteArray_AS_STRING(repr); - -done: - result = PyBytes_FromStringAndSize(PyByteArray_AS_STRING(repr), size); - Py_DECREF(repr); - return result; + size = p - PyBytes_AS_STRING(repr); + if (_PyBytes_Resize(&repr, size) < 0) + return NULL; + return repr; } static int -- cgit v0.12