UniversalCL icon indicating copy to clipboard operation
UniversalCL copied to clipboard

Memory leaks in popupmenus management

Open zizzo81 opened this issue 6 years ago • 3 comments

First of all you're doing a great work. I'm watching this library growing and it keeps getting better.

Compile the demo, assign the popupVert to the "Type some thing" edit, then run the demo and pop it up mantaining the mouse on the right side of the edit. Now without closing the popup menu, right click on the left side of the menu. The old popup will disappear and one identhical popup will appear.

No access violations. But leaving the demo you will be reported about a bunch of memory leaks.

zizzo81 avatar Nov 12 '19 23:11 zizzo81

An unexpected memory leak has occurred. The unexpected small block leaks are:

1 - 12 bytes: TObject x 1 21 - 28 bytes: TUPopupMenu.PopupForm_OnDeactive$ActRec x 1, TWinControlHelper.Animation$ActRec x 1 61 - 68 bytes: Unknown x 1 85 - 92 bytes: TIntAni x 1 133 - 140 bytes: TIntAni.UpdateFunction$ActRec x 1

zizzo81 avatar Nov 13 '19 13:11 zizzo81

I have noted this problem. This control should be completely rewritten instead of corrected. I have planned for this soon. Thanks for your support.

ghost avatar Nov 13 '19 13:11 ghost

I made a lot of changes to the TUPopupMenu overnight. As soon as I understand how to do a pull request (despite being a aged developer I'm a newbie here) I will share my changes with you so you can take the good things I've done and trash away anything you don't like.

zizzo81 avatar Nov 13 '19 13:11 zizzo81