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

Fix minor memory leak in Python module docstring handling

Browse source 
Noted in #582 by aurelj.
This commit is contained in:
Olly Betts 2016-01-12 13:49:55 +13:00
commit 584b328239
1 changed files with 8 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Source/Modules/python.cxx
View file

@ -797,12 +797,14 @@ public:
Swig_register_filebyname("shadow", f_shadow); Swig_register_filebyname("shadow", f_shadow);
Swig_register_filebyname("python", f_shadow); Swig_register_filebyname("python", f_shadow);
if (mod_docstring && Len(mod_docstring)) { if (mod_docstring) {
if (Len(mod_docstring)) {
const char *triple_double = "\"\"\""; const char *triple_double = "\"\"\"";
// follow PEP257 rules: https://www.python.org/dev/peps/pep-0257/ // follow PEP257 rules: https://www.python.org/dev/peps/pep-0257/
// reported by pep257: https://github.com/GreenSteam/pep257 // reported by pep257: https://github.com/GreenSteam/pep257
bool multi_line_ds = Strchr(mod_docstring, '\n') != 0; bool multi_line_ds = Strchr(mod_docstring, '\n') != 0;
Printv(f_shadow, triple_double, multi_line_ds ? "\n":"", mod_docstring, multi_line_ds ? "\n":"", triple_double, "\n\n", NIL); Printv(f_shadow, triple_double, multi_line_ds ? "\n":"", mod_docstring, multi_line_ds ? "\n":"", triple_double, "\n\n", NIL);
}
Delete(mod_docstring); Delete(mod_docstring);
mod_docstring = NULL; mod_docstring = NULL;
} }