On Jan 17, 2009, at 4:41 AM, Stefan Behnel wrote: > Robert Bradshaw wrote: >> It seems there's a bug in temp allocation, basically a temp is >> allocated but never released (even though it's freed). >> >> See http://hg.cython.org/cython-devel/file/4f0327bdebc9/tests/run/ >> call_crash.pyx > > The problem is that AttributeNode is not a NewTempExprNode and thus > fails > to pass on the request for freeing the temp. Changing its baseclass > fixes > this problem, but it also breaks tons of other test cases. It might > just be > one problem that kills all of them, but in any case, this needs some > investigation. I'll try to look into it this weekend if I find the > time.
Is this going to be a general problem with any inside a NewTempExprNode (plain) ExprNode? That could be bad. On that note, is there any reason that temp freeing is done as a separate step now instead of as part of the disposal code? (Someone more familiar with the new code probably has a trivial answer to this.) - Robert _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
