Memory leak

Hi,

I am not sure if this is the right place or not (and I apologise if it isn't) but the PAM has a memory leak (both version 1.5.2 and 1.6 beta).

When PAM reloads the application list, it calls TfrmMenu.ClearButtons, which clears the TAppListItem array. It doesn't actually frees the TAppListItem or TIcon objects. Upon exit, the TAppListItem array (and TIcon) objects are not freed.

There are two issues (with fixes):

1) TfrmMenu.ClearButtons needs to be modified from:

for intCounter:=0 to Length(arrAppItems) - 1 do
arrAppItems[intCounter].Icon:=nil;

to:

for intCounter:=0 to Length(arrAppItems) - 1 do
begin
arrAppItems[intCounter].Icon.Free; // Free the TIcon object
arrAppItems[intCounter].Icon:=nil;
arrAppItems[intCounter].Free; // Free the TAppListItem object
arrAppItems[intCounter]:=nil;
end;


P.S. I am just trying to improve PAM, not being critical...