mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-09-27 02:39:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 13

Make frames a PyVarObject. Use new GC API.

Browse source
This commit is contained in:
Neil Schemenauer 2001-08-29 23:52:17 +00:00
parent 01b66a80c4
commit 4f4817fee8
1 changed files with 14 additions and 30 deletions
Download patch file Download diff file Expand all files Collapse all files

44
Objects/frameobject.c
View file

@ -47,7 +47,7 @@ static struct getsetlist frame_getsetlist[] = {
f_back next item on free list, or NULL f_back next item on free list, or NULL
f_nlocals number of locals f_nlocals number of locals
f_stacksize size of value stack f_stacksize size of value stack
f_size size of localsplus ob_size size of localsplus
Note that the value and block stacks are preserved -- this can save Note that the value and block stacks are preserved -- this can save
another malloc() call or two (and two free() calls as well!). another malloc() call or two (and two free() calls as well!).
Also note that, unlike for integers, each frame object is a Also note that, unlike for integers, each frame object is a
@ -68,7 +68,7 @@ frame_dealloc(PyFrameObject *f)
PyObject **p; PyObject **p;
Py_TRASHCAN_SAFE_BEGIN(f) Py_TRASHCAN_SAFE_BEGIN(f)
PyObject_GC_Fini(f); _PyObject_GC_UNTRACK(f);
/* Kill all local variables */ /* Kill all local variables */
slots = f->f_nlocals + f->f_ncells + f->f_nfreevars; slots = f->f_nlocals + f->f_ncells + f->f_nfreevars;
fastlocals = f->f_localsplus; fastlocals = f->f_localsplus;
@ -125,7 +125,6 @@ frame_traverse(PyFrameObject *f, visitproc visit, void *arg)
for (p = f->f_valuestack; p < f->f_stacktop; p++) for (p = f->f_valuestack; p < f->f_stacktop; p++)
VISIT(*p); VISIT(*p);
} }
return 0; return 0;
} }
@ -171,8 +170,8 @@ PyTypeObject PyFrame_Type = {
PyObject_HEAD_INIT(&PyType_Type) PyObject_HEAD_INIT(&PyType_Type)
0, 0,
"frame", "frame",
sizeof(PyFrameObject) + PyGC_HEAD_SIZE, sizeof(PyFrameObject),
0, sizeof(PyObject *),
(destructor)frame_dealloc, /* tp_dealloc */ (destructor)frame_dealloc, /* tp_dealloc */
0, /* tp_print */ 0, /* tp_print */
0, /* tp_getattr */ 0, /* tp_getattr */
@ -188,7 +187,7 @@ PyTypeObject PyFrame_Type = {
PyObject_GenericGetAttr, /* tp_getattro */ PyObject_GenericGetAttr, /* tp_getattro */
PyObject_GenericSetAttr, /* tp_setattro */ PyObject_GenericSetAttr, /* tp_setattro */
0, /* tp_as_buffer */ 0, /* tp_as_buffer */
Py_TPFLAGS_DEFAULT | Py_TPFLAGS_GC, /* tp_flags */ Py_TPFLAGS_DEFAULT | Py_TPFLAGS_HAVE_GC,/* tp_flags */
0, /* tp_doc */ 0, /* tp_doc */
(traverseproc)frame_traverse, /* tp_traverse */ (traverseproc)frame_traverse, /* tp_traverse */
(inquiry)frame_clear, /* tp_clear */ (inquiry)frame_clear, /* tp_clear */
@ -241,34 +240,21 @@ PyFrame_New(PyThreadState *tstate, PyCodeObject *code, PyObject *globals,
if (builtins != NULL && !PyDict_Check(builtins)) if (builtins != NULL && !PyDict_Check(builtins))
builtins = NULL; builtins = NULL;
if (free_list == NULL) { if (free_list == NULL) {
/* PyObject_New is inlined */ f = PyObject_GC_NewVar(PyFrameObject, &PyFrame_Type, extras);
f = (PyFrameObject *)
PyObject_MALLOC(sizeof(PyFrameObject) +
extras*sizeof(PyObject *) +
PyGC_HEAD_SIZE);
if (f == NULL) if (f == NULL)
return (PyFrameObject *)PyErr_NoMemory(); return NULL;
f = (PyFrameObject *) PyObject_FROM_GC(f);
PyObject_INIT(f, &PyFrame_Type);
f->f_size = extras;
} }
else { else {
f = free_list; f = free_list;
free_list = free_list->f_back; free_list = free_list->f_back;
if (f->f_size < extras) { if (f->ob_size < extras) {
f = (PyFrameObject *) PyObject_AS_GC(f); f = PyObject_GC_Resize(PyFrameObject, f, extras);
f = (PyFrameObject *)
PyObject_REALLOC(f, sizeof(PyFrameObject) +
extras*sizeof(PyObject *) +
PyGC_HEAD_SIZE);
if (f == NULL) if (f == NULL)
return (PyFrameObject *)PyErr_NoMemory(); return NULL;
f = (PyFrameObject *) PyObject_FROM_GC(f);
f->f_size = extras;
} }
else else
extras = f->f_size; extras = f->ob_size;
PyObject_INIT(f, &PyFrame_Type); _Py_NewReference(f);
} }
if (builtins == NULL) { if (builtins == NULL) {
/* No builtins! Make up a minimal one. */ /* No builtins! Make up a minimal one. */
@ -323,8 +309,7 @@ PyFrame_New(PyThreadState *tstate, PyCodeObject *code, PyObject *globals,
f->f_valuestack = f->f_localsplus + (f->f_nlocals + ncells + nfrees); f->f_valuestack = f->f_localsplus + (f->f_nlocals + ncells + nfrees);
f->f_stacktop = f->f_valuestack; f->f_stacktop = f->f_valuestack;
_PyObject_GC_TRACK(f);
PyObject_GC_Init(f);
return f; return f;
} }
@ -486,7 +471,6 @@ PyFrame_Fini(void)
while (free_list != NULL) { while (free_list != NULL) {
PyFrameObject *f = free_list; PyFrameObject *f = free_list;
free_list = free_list->f_back; free_list = free_list->f_back;
f = (PyFrameObject *) PyObject_AS_GC(f); PyObject_GC_Del(f);
PyObject_DEL(f);
} }
} }