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

Memory leak #58

Open
atilaneves opened this issue Jul 19, 2018 · 2 comments
Open

Memory leak #58

atilaneves opened this issue Jul 19, 2018 · 2 comments
Labels
bug can't reproduce Cannot reproduce the issue

Comments

@atilaneves
Copy link
Collaborator

atilaneves commented Jul 19, 2018

@Excel() 
double[] foo()
{
    import core.memory : GC;
    GC.collect;
    GC.minimize;
    log(GC.stats.usedSize);
    auto ret = new double[](10_000_000);
    log(GC.stats.usedSize);
    return ret;
}

re-run in a single cell (so excel throws away old memory) and take a look at excel memory usage in task manager and the logged GC usage values

@atilaneves atilaneves added the bug label Jul 19, 2018
@atilaneves
Copy link
Collaborator Author

@John-Colvin I can't reproduce this. The memory amounts logged go up by a few hundred bytes every now and again (but not every time the formula is re-evaluated), then becomes stable.

@atilaneves
Copy link
Collaborator Author

@John-Colvin Oh, and I corrected new int in the description above to new double. Might that be why? Freeing a dynamic array of the wrong type?

@atilaneves atilaneves added the can't reproduce Cannot reproduce the issue label Jul 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug can't reproduce Cannot reproduce the issue
Projects
None yet
Development

No branches or pull requests

1 participant