Jump to content


Photo

2 small bugs (leak and vert scrollbar)

leak scrollbar

  • Please log in to reply
3 replies to this topic

#1 CDK

CDK
  • Members
  • 2 posts

Posted 15 February 2015 - 12:18 PM

Hi!

 

I'm using NxInspector 5.9.9.

 

 

1) Profiler says about memory leak in TNxCustomInspector.RedrawBorder at    

Theme := OpenThemeData(Handle, DataName);

Theme must be closed by CloseThemeData (?).

 

 

2) I'm creating about 50-70 invisible properties at startup:

PropItem := PropertyGrid.Items.AddChild(...);
....
PropItem.Visible := False;

And switch them on depending on the context. But vertical scroll bar seems to use the last item number for scrolling, ignoring their visibility (see attached image).

Attached Files



#2 Boki (Berg)

Boki (Berg)

    Boki (Berg)

  • Forum Admin
  • PipPipPipPipPip
  • 8,191 posts
  • Gender:Male

Posted 16 February 2015 - 05:35 AM

Hi,

 

I have fixed leak. I will include it in tomorrow update.

 

I will need couple of days to fix visibility problem. I hope that it is not hard.


boki@bergsoft.net | LinkedIn Profile
--
BergSoft Home Page: www.bergsoft.net
Users Section: users.bergsoft.net
Articles and Tutorials: help.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 CDK

CDK
  • Members
  • 2 posts

Posted 11 October 2015 - 12:51 PM

Hi Berg!

 

I tried to understand what the problem is with the vertical scroolbar and noticed that TNxCustomInspector.GetVisibleCount always returns too big value.

I also noticed that in most other parts of the code you use this verification:

if not FItems.Node[i].Hidden and FItems.Node[i].Visible

but in the GetVisibleCount only the first part of it.

 

Changes from

    if not FItems.Node[i].Hidden and

to

    if not FItems.Node[i].Hidden and FItems.Node[i].Visible and

fixes my problem.



#4 Boki (Berg)

Boki (Berg)

    Boki (Berg)

  • Forum Admin
  • PipPipPipPipPip
  • 8,191 posts
  • Gender:Male

Posted 12 October 2015 - 05:07 PM

Thank you,

 

Don't know how I miss it. I have applied into official code. Hope that will work fine now.


boki@bergsoft.net | LinkedIn Profile
--
BergSoft Home Page: www.bergsoft.net
Users Section: users.bergsoft.net
Articles and Tutorials: help.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.





Also tagged with one or more of these keywords: leak, scrollbar

0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users