App uses far too much memory

lastfm / lastfm-desktop

The official Last.fm desktop application suite

http://www.last.fm

GNU General Public License v3.0

462 stars 94 forks source link

App uses far too much memory #28

Closed RobFog closed 10 years ago

RobFog commented 11 years ago

Last.fm 2.1.35
Mac OS X 10.8.2
iTunes 11.0.2
Spotify 0.8.5.1333.g822e0de8

I've been noticing the newer app uses a lot more memory than the old version. Right now, it's sitting at 221MB Real Memory / 184,6MB Private Memory which seems unreasonably high. I vaguely remember the old version used something between 20 to 40MB Real Memory.

Please let me know if there's more information I can contribute.

eartle commented 11 years ago

Yep, this is an issue I really want to get around to fixing. Far too much memory is being used.

Benuuu commented 11 years ago

I just discovered this issue and this project. Any tips on where one might start looking to track down the leak? I've noticed that it allocates and doesn't free about one MB per track start.

RobFog commented 11 years ago

https://github.com/lastfm/lastfm-desktop/commit/778ab659263b4747fa098c54b03aa735709b6426

RobFog commented 11 years ago

Will this fix be incorporated into a release soon?

RobFog commented 10 years ago

Memory use seems to be greatly improved. Thanks! :-)