[patch_done] Memory leak on TZIBEventAlerter

Code patches written by our users to solve certain "problems" that were not solved, yet.

Moderators: gto, cipto_kh, EgonHugeist, mdaems

Post Reply
magnomp
Fresh Boarder
Fresh Boarder
Posts: 10
Joined: 08.01.2007, 17:39

[patch_done] Memory leak on TZIBEventAlerter

Post by magnomp »

After installing Zeos 6.6.3 and using it's TZIBEventAlerter I've noticed some memory leaking on my application.
This component instantiates a TStringList (FEvents) on it's constructor but never frees it.
I've added a call to FEvents.Free on TZIBEventAlerter destructor. Further, I've removed the following lines:
if Assigned(FEvents) then
TStringList(FEvents).OnChange := nil;
Because if FEvents is gonna be freed, there's no point on setting nil to OnChange.
User avatar
mdaems
Zeos Project Manager
Zeos Project Manager
Posts: 2766
Joined: 20.09.2005, 15:28
Location: Brussels, Belgium
Contact:

Post by mdaems »

Committed to testing branch. I'll port the change to 6.6-patches branch in about a week. SVN Rev. 425.

Mark
Image
User avatar
mdaems
Zeos Project Manager
Zeos Project Manager
Posts: 2766
Joined: 20.09.2005, 15:28
Location: Brussels, Belgium
Contact:

Post by mdaems »

Now available in the 6.6-patches branch.
Post Reply