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

Reduce memory consumption or provide a setting to do so #33

Open
mkelley33 opened this issue Jul 21, 2013 · 7 comments
Open

Reduce memory consumption or provide a setting to do so #33

mkelley33 opened this issue Jul 21, 2013 · 7 comments

Comments

@mkelley33
Copy link

I have 16 GB or RAM in my MBP. QuickHub is great, but I can't afford to devote 2 GB to it ;) One fix might be setting a limit on the memory consumption to something like 256 MB with the option of letting the user raise this limit. Hopefully it's not an obscure memory leak, but I thought you guys might like to know. Thanks!

@mkelley33
Copy link
Author

Also, I'm using version 1.6 from the AppStore. It was probably running for a little over a day before I noticed the memory usage.

@mkelley33
Copy link
Author

Console.app logs:

7/19/13 5:50:44.238 PM
QuickHub[1061]: Incorrect NSStringEncoding value 0x0000 detected.
Assuming NSASCIIStringEncoding. Will stop this compatiblity mapping behavior in the near future.

7/20/13 12:32:23.198 PM
QuickHub[1061]: Incorrect NSStringEncoding value 0x0000 detected.
Assuming NSASCIIStringEncoding. Will stop this compatiblity mapping behavior in the near future.

@chamerling
Copy link
Owner

Hi @mkelley33,

Yes there are some memory leaks, agree with you that this sort of app should not use so much. I will open source the app soon so that people can contribute and fix things ;)

@mkelley33
Copy link
Author

I look forward the open-sourced version, and maybe I'll be able to help fix some of the issues. In the meantime,
thank you and good luck!

@gwynantj
Copy link
Contributor

Hi @chamerling,

I've found this tool really helpful in the past.....so I've forked the app to start fixing the memory leaks......there are a lot of them.

Thanks

@chamerling
Copy link
Owner

Hi!

Thanks a lot, Will be nice to have a pull request ;)

Le 11 déc. 2013 à 18:38, Gwynant Jones [email protected] a écrit :

Hi @chamerling,

I've found this tool really helpful in the past.....so I've forked the app to start fixing the memory leaks......there are a lot of them.

Thanks


Reply to this email directly or view it on GitHub.

@gwynantj
Copy link
Contributor

I have proposed a pull request that fixes this. PR #38

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

No branches or pull requests

3 participants