Stowage/cpython
2
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-11-04 07:31:38 +00:00
Code Issues Projects Releases Packages Wiki Activity

Let marshal build-up sets and frozensets one element at a time.

Browse source 
Saves the unnecessary creation of a tuple as intermediate container.
This commit is contained in:
Raymond Hettinger 2008-01-26 08:37:28 +00:00
parent 08b50eb3d3
commit b423f02aa5
1 changed files with 8 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

18
Python/marshal.c
View file

@ -860,7 +860,7 @@ r_object(RFILE *p)
retval = NULL; retval = NULL;
break; break;
} }
v = PyTuple_New((int)n); v = (type == TYPE_SET) ? PySet_New(NULL) : PyFrozenSet_New(NULL);
if (v == NULL) { if (v == NULL) {
retval = NULL; retval = NULL;
break; break;
@ -875,18 +875,14 @@ r_object(RFILE *p)
v = NULL; v = NULL;
break; break;
} }
PyTuple_SET_ITEM(v, (int)i, v2); if (PySet_Add(v, v2) == -1) {
} Py_DECREF(v);
if (v == NULL) { Py_DECREF(v2);
retval = NULL; v = NULL;
break; break;
} }
if (type == TYPE_SET) }
v3 = PySet_New(v); retval = (v == NULL) ? NULL : v;
else
v3 = PyFrozenSet_New(v);
Py_DECREF(v);
retval = v3;
break; break;
case TYPE_CODE: case TYPE_CODE: