<SNIP> Hello Martin,
> Your patch seems 'free' memory allocated with 'new'. So either we > should 'm/calloc' the buffer or 'delete' it. I did ask Wiliam the same thing (I would have used delete), but the ntl wrapper is still C, so the calloc option might be the better one in the short term. Feel free to correct it and submit a patch. I am still getting the hang of the sage_lib Cython code, but now with the memory leak hunting things are definitely looking up. William also stated that there is a bug need to rewrite the ntl wrapper in C++ from scratch and those memleaks could give the needed push to get the rewrite started. > > Martin > Cheers, Michael > -- > 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/ -~----------~----~----~----~------~----~------~--~---
