diff options
author | Ken Hughes <khughes@pacific.edu> | 2005-11-30 11:18:06 +0300 |
---|---|---|
committer | Ken Hughes <khughes@pacific.edu> | 2005-11-30 11:18:06 +0300 |
commit | b7a4a6c8374e808837e45829ec85c48a409efd74 (patch) | |
tree | b6eb82e42fb0ff62bd60071267b4bf00afbb5a7f /source/blender/python/api2_2x/constant.c | |
parent | 00f43109705ab7f251e4487143049205e27e38f1 (diff) |
-- Bugfix #3186: Fix memory leaks caused by multiple calls to
PyDict_SetItemString() with objects that were not properly decrefed
afterwards. Due to the number of places this was used, I added a
wrapper EXPP_dict_set_item_str() to gen_utils.c to handle it.
This started as a scriptlink bug, due to how many times scripts were
being executed I think it just magnified how bad the memory leak in
BPy was. Animating the blend attached with this bug report would cause
memory to grow by about 3MB for every 280 frames. After the patch,
memory did not appear to grow at all (or at least not noticably using
Unix's ps and top utils).
Since many of the PyDict_SetItemString() calls were in initialization
modules I think my tests executed most of the changed code, but would
appreciate script users really giving feedback.
Diffstat (limited to 'source/blender/python/api2_2x/constant.c')
-rw-r--r-- | source/blender/python/api2_2x/constant.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/blender/python/api2_2x/constant.c b/source/blender/python/api2_2x/constant.c index 41a845184d4..1ebf50ffa55 100644 --- a/source/blender/python/api2_2x/constant.c +++ b/source/blender/python/api2_2x/constant.c @@ -74,7 +74,7 @@ static PyObject *new_const(void) "couldn't create constant object's dictionary")); } - return EXPP_incr_ret((PyObject *)constant); + return (PyObject *)constant; } //------------------------tp_doc //The __doc__ string for this object @@ -230,7 +230,7 @@ PyObject *PyConstant_New(void) //Inserts a key:value pair into the constant and then returns 0/1 int PyConstant_Insert(BPy_constant *self, char *name, PyObject *value) { - return PyDict_SetItemString(self->dict, name, value); + return EXPP_dict_set_item_str(self->dict, name, value); } //This is a helper function for generating constants...... PyObject *PyConstant_NewInt(char *name, int value) |