summaryrefslogtreecommitdiffstats
path: root/Parser/node.c
diff options
context:
space:
mode:
authorAndrew MacIntyre <andymac@bullseye.apana.org.au>2002-08-04 06:28:21 (GMT)
committerAndrew MacIntyre <andymac@bullseye.apana.org.au>2002-08-04 06:28:21 (GMT)
commit80d4e2acf58433fb2e84ee55d183a7995dc44d0e (patch)
treee4c33fd6d334c9f5bf5fd5260249fd650202c445 /Parser/node.c
parent4104db39b8c8a85b884bbba051d21b7ea1d21ee1 (diff)
downloadcpython-80d4e2acf58433fb2e84ee55d183a7995dc44d0e.zip
cpython-80d4e2acf58433fb2e84ee55d183a7995dc44d0e.tar.gz
cpython-80d4e2acf58433fb2e84ee55d183a7995dc44d0e.tar.bz2
SF patch #578297:
Change the parser and compiler to use PyMalloc. Only the files implementing processes that will request memory allocations small enough for PyMalloc to be a win have been changed, which are:- - Python/compile.c - Parser/acceler.c - Parser/node.c - Parser/parsetok.c This augments the aggressive overallocation strategy implemented by Tim Peters in PyNode_AddChild() [Parser/node.c], in reducing the impact of platform malloc()/realloc()/free() corner case behaviour. Such corner cases are known to be triggered by test_longexp and test_import. Jeremy Hylton, in accepting this patch, recommended this as a bugfix candidate for 2.2. While the changes to Python/compile.c and Parser/node.c backport easily (and could go in), the changes to Parser/acceler.c and Parser/parsetok.c require other not insignificant changes as a result of the differences in the memory APIs between 2.3 and 2.2, which I'm not in a position to work through at the moment. This is a pity, as the Parser/parsetok.c changes are the most important after the Parser/node.c changes, due to the size of the memory requests involved and their frequency.
Diffstat (limited to 'Parser/node.c')
-rw-r--r--Parser/node.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/Parser/node.c b/Parser/node.c
index 780c230..75900ce 100644
--- a/Parser/node.c
+++ b/Parser/node.c
@@ -7,7 +7,7 @@
node *
PyNode_New(int type)
{
- node *n = PyMem_NEW(node, 1);
+ node *n = (node *) PyObject_MALLOC(1 * sizeof(node));
if (n == NULL)
return NULL;
n->n_type = type;
@@ -92,7 +92,8 @@ PyNode_AddChild(register node *n1, int type, char *str, int lineno)
return E_OVERFLOW;
if (current_capacity < required_capacity) {
n = n1->n_child;
- PyMem_RESIZE(n, node, required_capacity);
+ n = (node *) PyObject_REALLOC(n,
+ required_capacity * sizeof(node));
if (n == NULL)
return E_NOMEM;
n1->n_child = n;
@@ -116,7 +117,7 @@ PyNode_Free(node *n)
{
if (n != NULL) {
freechildren(n);
- PyMem_DEL(n);
+ PyObject_FREE(n);
}
}
@@ -127,7 +128,7 @@ freechildren(node *n)
for (i = NCH(n); --i >= 0; )
freechildren(CHILD(n, i));
if (n->n_child != NULL)
- PyMem_DEL(n->n_child);
+ PyObject_FREE(n->n_child);
if (STR(n) != NULL)
- PyMem_DEL(STR(n));
+ PyObject_FREE(STR(n));
}