jyapayne/swig
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Cleanup accessing/decref of globals, to avoid code bloat in init function.

Browse source
This commit is contained in:
Andrew Rogers 2018-11-27 23:30:51 +00:00
commit 0fecd1538f
2 changed files with 13 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Lib/python/pyinit.swg
View file

@ -227,11 +227,7 @@ SWIG_Python_addvarlink(PyObject *p, const char *name, PyObject *(*get_attr)(void
SWIGINTERN PyObject * SWIGINTERN PyObject *
SWIG_globals(void) { SWIG_globals(void) {
static PyObject *_SWIG_globals = 0; static PyObject *_SWIG_globals = 0;
if (!_SWIG_globals) { if (!_SWIG_globals) _SWIG_globals = SWIG_newvarlink();
_SWIG_globals = SWIG_newvarlink();
} else {
Py_INCREF(_SWIG_globals);
}
return _SWIG_globals; return _SWIG_globals;
} }

25
Source/Modules/python.cxx
View file

@ -3316,21 +3316,21 @@ public:
getf = NewWrapper(); getf = NewWrapper();
setf = NewWrapper(); setf = NewWrapper();
Printf(f_init, "\t globals = SWIG_globals();\n");
Printf(f_init, "\t if (!globals) {\n");
Printf(f_init, " PyErr_SetString(PyExc_TypeError, \"Failure to create SWIG globals.\");\n");
Printf(f_init, "#if PY_VERSION_HEX >= 0x03000000\n");
Printf(f_init, "\t return NULL;\n");
Printf(f_init, "#else\n");
Printf(f_init, "\t return;\n");
Printf(f_init, "#endif\n");
Printf(f_init, "\t }\n");
/* If this is our first call, add the globals variable to the /* If this is our first call, add the globals variable to the
Python dictionary. */ Python dictionary. */
if (!have_globals) { if (!have_globals) {
Printf(f_init, "\t globals = SWIG_globals();\n");
Printf(f_init, "\t if (!globals) {\n");
Printf(f_init, " PyErr_SetString(PyExc_TypeError, \"Failure to create SWIG globals.\");\n");
Printf(f_init, "#if PY_VERSION_HEX >= 0x03000000\n");
Printf(f_init, "\t return NULL;\n");
Printf(f_init, "#else\n");
Printf(f_init, "\t return;\n");
Printf(f_init, "#endif\n");
Printf(f_init, "\t }\n");
Printf(f_init, "\t PyDict_SetItemString(md, \"%s\", globals);\n", global_name); Printf(f_init, "\t PyDict_SetItemString(md, \"%s\", globals);\n", global_name);
Printf(f_init, "\t Py_DECREF(globals);\n");
if (builtin) if (builtin)
Printf(f_init, "\t SwigPyBuiltin_AddPublicSymbol(public_interface, \"%s\");\n", global_name); Printf(f_init, "\t SwigPyBuiltin_AddPublicSymbol(public_interface, \"%s\");\n", global_name);
have_globals = 1; have_globals = 1;
@ -3418,12 +3418,11 @@ public:
Wrapper_print(getf, f_wrappers); Wrapper_print(getf, f_wrappers);
/* Now add this to the variable linking mechanism */ /* Now add this to the variable linking mechanism */
Printf(f_init, "\t SWIG_addvarlink(globals,\"%s\",%s, %s);\n", iname, vargetname, varsetname); Printf(f_init, "\t SWIG_addvarlink(globals, \"%s\", %s, %s);\n", iname, vargetname, varsetname);
if (builtin && shadow && !assignable && !in_class) { if (builtin && shadow && !assignable && !in_class) {
Printf(f_init, "\t PyDict_SetItemString(md, \"%s\", PyObject_GetAttrString(SWIG_globals(), \"%s\"));\n", iname, iname); Printf(f_init, "\t PyDict_SetItemString(md, \"%s\", PyObject_GetAttrString(globals, \"%s\"));\n", iname, iname);
Printf(f_init, "\t SwigPyBuiltin_AddPublicSymbol(public_interface, \"%s\");\n", iname); Printf(f_init, "\t SwigPyBuiltin_AddPublicSymbol(public_interface, \"%s\");\n", iname);
} }
Printf(f_init, "\t Py_DECREF(globals);\n");
Delete(vargetname); Delete(vargetname);
Delete(varsetname); Delete(varsetname);
Delete(getname); Delete(getname);