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 a memory leak #69

Merged
merged 1 commit into from
Jan 24, 2022
Merged

fix a memory leak #69

merged 1 commit into from
Jan 24, 2022

Commits on Jan 22, 2022

  1. fix a memory leak

    Memory usage of improviz would increase linearly over
    time until crash from memory exhaustion.
    
    deepseq is a bit of a sledgehammer,
    probably a more precise intervention is possible.
    
    The key change is the ... $! force newGe
    which makes the data in the TVar be stored strictly.
    
    Otherwise (conjecture) the data refers to previous
    copies of the data structures, etc, which leaks
    space because the whole data structure must be
    retained.
    claudeha committed Jan 22, 2022
    Configuration menu
    Copy the full SHA
    c99e7a7 View commit details
    Browse the repository at this point in the history