MSNPSharp 4.0 BETA2 using too much RAM

kythyria / msnp-sharp

Automatically exported from code.google.com/p/msnp-sharp

0 stars 0 forks source link

MSNPSharp 4.0 BETA2 using too much RAM #256

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What steps will reproduce the problem?
MSNPSharp 4.0 BETA2  using too much RAM

What version of the product are you using? (MSNPSharp, OS, Mono etc.)
MSNPSharp 4.0 BETA2, Windows 7 x64, .NET 2.0

Original issue reported on code.google.com by jarvis.chen66@gmail.com on 10 Feb 2011 at 8:44

Attachments:

未命名.png

GoogleCodeExporter commented 9 years ago

I found the problem. In Contact.Initialized, calling these functions raises 1GB 
RAM;

displayImage = DisplayImage.CreateDefaultImage(Mail);
sceneImage = SceneImage.CreateDefaultImage(Mail);

Original comment by hepha...@gmail.com on 10 Feb 2011 at 10:41

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r2013.

Original comment by hepha...@gmail.com on 10 Feb 2011 at 11:43

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Please check out svn and try again.
RAM has reduced to 100KB from 1GB :)

Original comment by hepha...@gmail.com on 10 Feb 2011 at 11:47

Added labels: Milestone-Release4.0.0

GoogleCodeExporter commented 9 years ago

How many contacts do you have? Did you test on 3.1 and have the same problem?

Original comment by freezing...@gmail.com on 10 Feb 2011 at 8:56

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

About 600 peoples.
3.1 work normally.

Original comment by jarvis.chen66@gmail.com on 11 Feb 2011 at 2:25

GoogleCodeExporter commented 9 years ago

Hi, can you try the current trunk version and check whether the problem still 
arise?

Original comment by freezing...@gmail.com on 12 Feb 2011 at 8:20

GoogleCodeExporter commented 9 years ago

Pang,
To repoduce the issue:
- Download beta2 release
- Login with testmsnpsharp@live account (it have about 400 contacts)
- You will see 1GB ram usage.

- Now update svn and login again.
- RAM usage is 35MB.

Anyway, I have just merged into 3.1 stable and Now 3.1 stable saves 150MB RAM 
if you have about 500 contacts.

Original comment by hepha...@gmail.com on 12 Feb 2011 at 9:53

Changed state: Verified
Added labels: Milestone-Release3.1.3
Removed labels: Milestone-Release4.0.0

GoogleCodeExporter commented 9 years ago

Hi Ehtem,

I don't think this issue should be closed this fast since the user already 
verify the performance for 3.1, I don't know why he say 3.1 works normally. If 
he used a modified client (If that client ) that might mean our 4.0 has other 
potential memory leak.

Original comment by freezing...@gmail.com on 12 Feb 2011 at 10:34

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

Actually what I want to say is, you make a good fix, but we need the user to 
confirm it is the problem because I really not sure what exactly cause it.

Original comment by freezing...@gmail.com on 12 Feb 2011 at 10:37

GoogleCodeExporter commented 9 years ago

The current trunk version work normally.

Original comment by jarvis.chen66@gmail.com on 12 Feb 2011 at 1:30

GoogleCodeExporter commented 9 years ago

ok, good job Ethem :)

Original comment by freezing...@gmail.com on 12 Feb 2011 at 8:39

Changed state: Fixed