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 #12

Closed
wants to merge 1 commit into from
Closed

Conversation

Isopod
Copy link

@Isopod Isopod commented Feb 18, 2021

With every keystroke the program creates a bunch of objects which are never being freed. Pascal is not a garbage-collected language, so these objects will pile up and the process will eventually run out of memory. This is not good.

Here's my attempt at fixing this. I can't guarantee that I found every leak, but at least it should be an improvement.

For the future, I would recommend getting into the habit of writing the cleanup code immediately. It's a lot harder to identify those kinds of leaks later and you are much more likely to overlook something.

Closes #2

@Isopod Isopod closed this Oct 3, 2022
@Isopod Isopod deleted the memory-management branch October 3, 2022 00:52
@Isopod Isopod restored the memory-management branch November 10, 2022 05:53
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.

memory management
1 participant