[issue3668] "s*" argument parser marker leaks memory

Amaury Forgeot d'Arc report at bugs.python.org
Mon Aug 25 14:57:08 CEST 2008


Amaury Forgeot d'Arc <amauryfa at gmail.com> added the comment:

This patch elegantly reuses the existing cleanup list.

Only two remarks: 
- there are a few tabs/spaces inconsistencies.
- I would make the cleanup_ptr explicit:
intead of 
      addcleanup(*buffer, freelist, NULL);
I'd prefer
      addcleanup(*buffer, freelist, cleanup_ptr);

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue3668>
_______________________________________


More information about the Python-bugs-list mailing list