jake-phy / WindowIconList

GNU General Public License v2.0
75 stars 26 forks source link

"cinnamon --replace" memory usage too high #62

Open harry218 opened 9 years ago

harry218 commented 9 years ago

Hi, I just have this issue when using the version from git. It can take up to 2GB after a few minutes of using this applet. I don't have this problem with the stable version from Cinnamon website. I also don't have many applets enabled and this is the only third-party applet I'm using right now. The Cinnamon version is 2.2.16 on Arch. Thanks.

jake-phy commented 9 years ago

So your saying it is taking up two GB of memory? Thanks for reporting I will look into it.

-----Original Message----- From: "harry218" notifications@github.com Sent: ‎10/‎21/‎2014 1:44 AM To: "jake-phy/WindowIconList" WindowIconList@noreply.github.com Subject: [WindowIconList] "cinnamon --replace" memory usage too high. (#62)

Hi, I just have this issue when using the version from git. It can take up to 2GB after a few minutes of using this applet. I don't have this problem with the stable version from Cinnamon website. I also don't have many applets enabled and this is the only third-party applet I'm using right now. The Cinnamon version is 2.2.16 on Arch. Thanks. — Reply to this email directly or view it on GitHub.=

ghost commented 9 years ago

I don't think that this issue is related at all with this applet: https://github.com/linuxmint/Cinnamon/issues/3635

harry218 commented 9 years ago

Thanks, it could be the Cinnamon issue itself. I don't have much experience in using other Cinnamon third party applets.

ghost commented 9 years ago

http://cinnamon-spices.linuxmint.com/applets/view/203

ghost commented 9 years ago

Destroy the cairo context on an explicit way: Gnome shell code (thanks to @JosephMcc): https://github.com/GNOME/gnome-shell/commit/9548cd834156447ef8a36a038f0b48eb8992fa8b#diff-09304ba2858e4c3eefca50fd13f45308R564

Cinnamon relate issue: https://github.com/linuxmint/Cinnamon/issues/3690#issuecomment-68844977

jake-phy commented 9 years ago

Is this an issue with my applet? I wasn't able to find any area where I am explicitly using cario.

ghost commented 9 years ago

Where do you render your special buttons? All popup menu are also render over a cairo surface, to see the arrow... Also i see that when you destroy a lot of memory, some time gc can not handled all this...

jake-phy commented 9 years ago

Hmm ok so I have to destroy menu's cario context manually?

ghost commented 9 years ago

I don't know i also don't have any proved that this really occurs and this are not apply on cinnamon, but clear your applet use a lot of this so if there are a real problem will be detected on your applet first than on cinnamon... I think...

jake-phy commented 9 years ago

Ok i'll keep a lookout for it.