Memory Leak?

[Razor]

Hello!

Yes i get error when closing my application,I included the screenshot.But this error and message only show when i use FastMM(Memory Manager replacement). :wall:


Can you give me an advice on making my application faster,it takes about 3-4 secs to start?

[DGL-luke]

oh yeah, fastmm tells you that you have A WHOLE BUNCH of mermory leaks. You have to free all object instances you use after use.

As they mainly seem to be VCL components, maybe you are dynamically instatiating them? then you should still free them after use!

[Razor]

So you mean like this?

Delphi-Quellcode:

procedure tmainfrm.cpuid;
begin
 MiTeC_CPU1.RefreshData;
 RzPanel10.caption:=MiTeC_CPU1.SerialNumber;
 if mitec_cpu1.CPUIDSupported=true then
 RzPanel11.Caption:='Yes'
 else
 RzPanel11.Caption:='No';
end;
MiTeC_CPU1.free;
 RzPanel10.caption.free;

end;

[Phoenix]

Actually the how is correct. But the where or better: when is wrong.

Every control you create dynamically, you need to free in the destructor of the owner. That is - in that case - the destructor of the Form.