[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-25 Thread Thomas Heller
Thomas Heller [EMAIL PROTECTED] added the comment: Can you please elaborate your (apparent) concerns about this patch? IOW, why did you not check it in? I have no concerns about the patch, and I am currently committing it. I'm uploading so that I can points others to it, and (hopefully) to

[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-25 Thread Martin v. Löwis
Martin v. Löwis [EMAIL PROTECTED] added the comment: Martin, are there any problems with this approach? No, it's fine. I was just puzzled. __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue2682 __

[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-25 Thread Thomas Heller
Thomas Heller [EMAIL PROTECTED] added the comment: Also fixed in release25-maint, rev 62504. -- resolution: accepted - fixed status: open - closed __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue2682 __

[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-24 Thread Thomas Heller
New submission from Thomas Heller [EMAIL PROTECTED]: Zachary Pincus posted a message about this cyclic reference in ctypes CFunctionType objects. The reference has the problem that these objects are cleaned up later than expected. The attached patch fixes this problem by removing the cyclic

[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-24 Thread Martin v. Löwis
Martin v. Löwis [EMAIL PROTECTED] added the comment: Can you please elaborate your (apparent) concerns about this patch? IOW, why did you not check it in? -- nosy: +loewis __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue2682

[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-24 Thread Thomas Heller
Thomas Heller [EMAIL PROTECTED] added the comment: Can you please elaborate your (apparent) concerns about this patch? IOW, why did you not check it in? I have no concerns about the patch, and I am currently committing it. I'm uploading so that I can points others to it, and (hopefully) to

[issue2682] cyclic reference in ctypes CFunctionType objects

2008-04-24 Thread Thomas Heller
Thomas Heller [EMAIL PROTECTED] added the comment: Fixed in trunk and py3k, committed as rev 62481 (trunk), rev 62484 (py3k). I'll leave this open until the decision about backporting to release25-maint is made. -- resolution: - accepted __ Tracker