Jump to content


Photo

A little memory leak


  • Please log in to reply
6 replies to this topic

#1 karl

karl
  • Members
  • 52 posts
  • Gender:Male
  • Location:Austria

Posted 04 December 2015 - 02:08 AM

Hi Boki,
 
I just discovered a little Memory leak in nextinspector6.
 
If one moves the mouse over a node and there is not enough space to diplay the full NodeCaption, (so the node caption is diplayed eg with ellipsis)
 
this white box, showing the whole node-caption text appears right next to the cursor. (I don't know how its called)
 
This somehow leaks memory. (I always monitor my applications memory usage)
 
Please investigate.
 
Thanks, Karl.

#2 Boki (Berg)

Boki (Berg)

    Boki (Berg)

  • Forum Admin
  • PipPipPipPipPip
  • 7,834 posts
  • Gender:Male

Posted 04 December 2015 - 02:33 AM

I will Karl,

I am using FastMm4. Not sure how I miss it. Which one you use?
boki@bergsoft.net | LinkedIn Profile
--
BergSoft Home Page: www.bergsoft.net
Members Section: bms.bergsoft.net
Articles and Tutorials: dn.bergsoft.net (Developers Network)
--
BergSoft Facebook page
--
Send us applications made with our components and we will submit them on: www.bergsoft.net/apps.htm. Link to this page will be also set on home page too.

#3 karl

karl
  • Members
  • 52 posts
  • Gender:Male
  • Location:Austria

Posted 04 December 2015 - 03:39 AM

I now also checked with fastmm, and youre right, it's not detected.
 
It's clearly there becouse in my apps I always show the currently used memory in a memo and you can see it go up when mouseover a nextinspector node
and then not going down again. You can also see it if you look at the used application memory in windows task manager.
 
Probably it's not detected becouse the leaking memory seems to be freed after the form containing nextInspector is freed.
 
Maybe it's something like elements beeing indefinitely added to a list or something similar, so rather a bug then a memory leak, fastmm won't detect this.
Still, this is not very nice if you keep the form for a long time.

#4 karl

karl
  • Members
  • 52 posts
  • Gender:Male
  • Location:Austria

Posted 05 December 2015 - 12:54 AM

Hi Boki,

Any updates on this issue?

It would be really nice to have it fixed.

You can easily reproduce it just add some nodes to Nextinspector, then move the mouse over them.
You will see your app memory usage going up and not going down again if you monitor it.

Thanks, Karl.

#5 Boki (Berg)

Boki (Berg)

    Boki (Berg)

  • Forum Admin
  • PipPipPipPipPip
  • 7,834 posts
  • Gender:Male

Posted 05 December 2015 - 02:53 AM

Hi Karl,

I hope that I will fix it tomorrow.
boki@bergsoft.net | LinkedIn Profile
--
BergSoft Home Page: www.bergsoft.net
Members Section: bms.bergsoft.net
Articles and Tutorials: dn.bergsoft.net (Developers Network)
--
BergSoft Facebook page
--
Send us applications made with our components and we will submit them on: www.bergsoft.net/apps.htm. Link to this page will be also set on home page too.

#6 karl

karl
  • Members
  • 52 posts
  • Gender:Male
  • Location:Austria

Posted 05 December 2015 - 02:20 PM

Hi Boki,

Please keep me up to date.

Also regarding the new stable version of nextgrid6 with the rowheight-event.


Thank you for your effort.

#7 karl

karl
  • Members
  • 52 posts
  • Gender:Male
  • Location:Austria

Posted 05 December 2015 - 06:47 PM

Hi Boki,

Thank you for fixing the memory leak so quickly :).

Also onMeasureRowHeight works fine.

Regards, Karl.




0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users