On Wednesday 29 August 2007, William Stein wrote: > On 8/29/07, Martin Albrecht <[EMAIL PROTECTED]> wrote: > > It seems that none of those is a memleak because they all operate on a > > Python objects only which should be taken care of py Cython. > > What does this sentence refer to? The ntl wrapper stuff? Something else? > The ntl wrapper stuff is definitely a memleak -- to fix it we just need to > write a function that converts a char* to a Python string and deletes the > char*'s memory. If you're referring to some other Cython code, sometimes > it will leak if really awkward exception handling is being used -- in such > cases the code must be rewritten.
It refers to the massive list of potential memleaks Michael posted earlier: <SNIP> build/sage/matrix/strassen.pyx: return str(self._intervals) build/sage/ext/multi_modular.pyx: return str(type(self)) +str(self.list()) build/sage/groups/perm_gps/permgroup_element.py: return str(self) build/sage/groups/matrix_gps/matrix_group_element.py: return str(self.__mat) build/sage/algebras/algebra_order_element.py: return str(self.__ambient_algebra_element) <SNIP> AFAIK the NTL wrapper stuff has been rewritten, there is a patch by mabshoff for ntl.pyx and a patch by me for ntl_wrap.cc on trac. Martin -- name: Martin Albrecht _pgp: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x8EF0DC99 _www: http://www.informatik.uni-bremen.de/~malb _jab: [EMAIL PROTECTED] --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/ -~----------~----~----~----~------~----~------~--~---
