Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix memory leaks #97

Merged
merged 1 commit into from Feb 25, 2019
Merged

Fix memory leaks #97

merged 1 commit into from Feb 25, 2019

Conversation

MartinGuehmann
Copy link
Collaborator

I found some leaks, the one in CityData was introduced by redesigning the CityData class. And the one of RadarMap only showed up after opening the Gaia Controller screen.
..\ctp2_code\gs\gameobj\CityData.cpp
..\ctp2_code\ui\aui_ctp2\radarmap.cpp

I found some leaks, the one in CityData was introduced by redesigning the CityData class. And the one of RadarMap only showed up after opening the Gaia Controller screen.
..\ctp2_code\gs\gameobj\CityData.cpp
..\ctp2_code\ui\aui_ctp2\radarmap.cpp
@LynxAbraxas
Copy link
Contributor

Seems reasonable, but did not test yet with ctp2DF. @MartinGuehmann How did you notice those leaks?

@MartinGuehmann
Copy link
Collaborator Author

Seems reasonable, but did not test yet with ctp2DF. @MartinGuehmann How did you notice those leaks?

The debug version generates on shut-down two files in the working directory, by default the executable directory: CTP_LEAKS_99999.TXT and CTP_LEAKS_ALT_99999.TXT.

I should have seen the one in CityData earlier. The one from the Gaia controller screen you would only see if you have opened the Gaia controller screen, which is something you don't see so often.

@MartinGuehmann
Copy link
Collaborator Author

This has been long enough around so that it can also go in.

@MartinGuehmann MartinGuehmann merged commit f8f1c59 into civctp2:master Feb 25, 2019
@MartinGuehmann MartinGuehmann deleted the FixLeaks branch February 25, 2019 19:53
@LynxAbraxas
Copy link
Contributor

LynxAbraxas commented Feb 25, 2019

@MartinGuehmann, sorry, I'm currently not able to keep up with testing all your cool contributions. With #84 you should now get basic feedbacks similar to those you got from me, except for the manual testing of the game run from the DI, which you can also do yourself, see #84 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants