memory leak on windows - Githubissues

mnba / shedskin

Automatically exported from code.google.com/p/shedskin

0 stars 0 forks source link

memory leak on windows #127

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What steps will reproduce the problem?
1. shedskin -e fysphun & make
2. run fysphun_main.py
3.

What is the expected output? What do you see instead?
use process Explorer can find, python.exe's memory is raising continually.

What version of the product are you using? On what operating system?
shedskin0.7

Please provide any additional information below.

Original issue reported on code.google.com by seewin...@gmail.com on 24 Jan 2011 at 2:39

GoogleCodeExporter commented 9 years ago

thanks for reporting! I noticed this problem before (not under Windows), and 
thought I fixed it.. I will have a look at this hopefully in the next few days. 
thanks again!

Original comment by mark.duf...@gmail.com on 24 Jan 2011 at 6:41

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

well, it looks like I fixed the leakage for fysphun (if you'd like to try: 
http://gitorious.org/shedskin). I will try to perform some tests later this 
week to see if this doesn't cause other problems, and to see if I can find 
related leaks..

Original comment by mark.duf...@gmail.com on 24 Jan 2011 at 8:21

GoogleCodeExporter commented 9 years ago

thanks, it is fix when I use gitorious's shedskin.

Original comment by seewin...@gmail.com on 25 Jan 2011 at 10:25

GoogleCodeExporter commented 9 years ago

thanks for testing ;) I think I introduced a problem for other programs though. 
I will have a good look at a more general fix hopefully tomorrow.

Original comment by mark.duf...@gmail.com on 25 Jan 2011 at 11:40

GoogleCodeExporter commented 9 years ago

alright, found the real problem now.. fixed it in GIT, and added a refcount 
test script that I will run before each release. thanks for triggering!

Original comment by mark.duf...@gmail.com on 26 Jan 2011 at 11:04

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

btw increasing the test coverage of this script uncovered a few other leaks. I 
will fix these shortly..

Original comment by mark.duf...@gmail.com on 26 Jan 2011 at 11:22

GoogleCodeExporter commented 9 years ago

okay, fixed those as well, and verified that the fysphun, circle and c64 
examples work without leaks. if you'd be interested, here's the new refcount 
test script:

http://gitorious.org/shedskin/mainline/trees/master/tests/leak

Original comment by mark.duf...@gmail.com on 26 Jan 2011 at 11:58