🚀 go-pugleaf

Somewhere in your code, you need a:

delete psaStr;

--
-------------------------------------------------------------------------
Free software  - Baxter Codeworks  www.baxcode.com
-------------------------------------------------------------------------


"David Ladanyi" <david_ladanyi@yahoo.com> wrote in message
news:30890494.0210062256.7a8a8fa@posting.google.com...
> Hi
>
> I have a program that seems to run fine in the emulator except that
> whenever I try to close the program I'm told that I have a memory
> leak. I've figured out what statement is causing the leak but I can't
> figure out how to fix it. This code is called when a certain form is
> being initialized:
>
> double psaVal = pronData->pretx_psa;
> char* psaStr = new char[6];
>
> if(psaVal != NO_VALUE) {
> doubleToStr(psaStr, psaVal, 1);
> }
> else
> psaStr = "";
>
> SetFieldHandle(GetObjectPtr<FieldType>( PreTxPSAField), psaStr);
>
> This code simply creates a string from a double and sets a field to
> that string. The memory leak was caused by the "new char[6]" line so I
> put in a deallocation function that would be called when the form
> receives a frmCloseEvent (I made sure to set handled=false):
>
> void PreTxCleanUp() {
>
>         FormType* frmP = FrmGetActiveForm();
> FieldType* field = (FieldType*)
> FrmGetObjectPtr(frmP,FrmGetObjectIndex (frmP, PreTxPSAField));
>
> MemHandle refs = FldGetTextHandle(field);
>
> if(refs != NULL) {//  && MemHandleUnlock(refs) != memErrInvalidParam)
> {
> FldSetTextHandle (field, NULL);
> MemHandleFree(refs);
> }
>
> }
>
> This function doesnt give any errors buy it also doesn't fix the
> problem. Every time I open and close the form in question it creates a
> leaked memory chunk. Am I not de-allocating the memory properly or is
> there a problem with the emulator? Also, if the memory leak is caused
> by a variable local to a certain form why isn't the emulator telling
> me about the leak as soon as I close the form as opposed to waiting
> until I close the whole program? Thanks in advance for any advice.
>
> Dave