Disposing of objects

	Bottom Previous Topic Next Topic
Register To Post

mritter0

Posted on: 2016/8/29 2:49 #1

Just popping in

Is there a preferred way to check if an object is allocated before disposing? I just don't think all of mine are being disposed of.


[allocate node]

LocalContentNode->DefaultImage=NULL;

--- OR ---

LocalContentNode->DefaultImage=ZERO;





VOID

FreeContentNode(struct ContentNode *LocalContentNode)

{

    if (LocalContentNode->DefaultImage != NULL)

    {

        DisposeImage(LocalContentNode->DefaultImage);

        LocalContentNode->DefaultImage=NULL;

    }

    if (LocalContentNode->Text1Label != NULL)

    {

        IIntuition->DisposeObject(LocalContentNode->Text1Label);

        LocalContentNode->Text1Label=NULL;

    }

    if (LocalContentNode->Text2Label != NULL)

    {

        IIntuition->DisposeObject(LocalContentNode->Text2Label);

        LocalContentNode->Text2Label=NULL;

    }

}



--- OR ---



VOID

FreeContentNode(struct ContentNode *LocalContentNode)

{

    if (LocalContentNode->DefaultImage != ZERO)

    {

        DisposeImage(LocalContentNode->DefaultImage);

        LocalContentNode->DefaultImage=ZERO;

    }

    if (LocalContentNode->Text1Label != ZERO)

    {

        IIntuition->DisposeObject(LocalContentNode->Text1Label);

        LocalContentNode->Text1Label=ZERO;

    }

    if (LocalContentNode->Text2Label != ZERO)

    {

        IIntuition->DisposeObject(LocalContentNode->Text2Label);

        LocalContentNode->Text2Label=ZERO;

    }

}

broadblues

Re: Disposing of objects

Posted on: 2016/8/29 3:25 #2

Home away from home

NULL is ((APTR)0L)

where as

ZERO id ((BPTR)0L)

So you should use NULL because you dealing with C pointers not BCPL pointers.

ZERO is for locks and file handles.

I would think the code would emits warnings if the wrong one were used...

What makes you think that your objects are being dispoed of?

Blender For OS4.x : Blues : Walker Broad

mritter0

Re: Disposing of objects

Posted on: 2016/8/29 3:43 #3

Just popping in

I was using NULL.

I have a memory leak I can't track down. It is much smaller when I don't use images, so....testing all possible options.

Workbench Explorer - A better way to browse drawers

broadblues

Re: Disposing of objects

Posted on: 2016/8/29 11:19 #4

Home away from home

@mritter0

How do you determine the memeory leek?

If it by apperent reduction in memeory available in the workbnch title bar, or shown by Avail, then it may not be a leek rather cached data. Bitmap.image caches data for example.

Try running 'expunge all' (should be in the SDK) and see if you get it back.

If your label images have embedded images, then remeber they need to be disposed of sperately unless you specify LABEL_DisposeImage before addibng the image to the label. This is off by default.

Blender For OS4.x : Blues : Walker Broad

mritter0

Re: Disposing of objects

Posted on: 2016/8/29 22:43 #5

Just popping in

Good call! Thanks.

I was hoping it was something like that, and not something I was missing.

Is there a way to have it not cache some of the images? Some are very short term and is a "waste" to cache them.

Workbench Explorer - A better way to browse drawers

mritter0

Re: Disposing of objects

Posted on: 2016/10/5 3:25 #6

Just popping in

Is there a way to not cache the images? They are very short term and is a waste to cache them.


Img=IIntuition->NewObject(BitMapClass,NULL,

    BITMAP_SourceFile,                        "PROGDIR:Images/24x1",

    BITMAP_SelectSourceFile,                "PROGDIR:Images/24x1",

    BITMAP_DisabledSourceFile,                "PROGDIR:Images/24x1",

    BITMAP_Screen,                            DefaultPubScreen,

    BITMAP_Precision,                        PRECISION_EXACT,

    BITMAP_Masking,                            TRUE,



        BITMAP_DoNotCache,        TRUE,



TAG_DONE);

Some sort of tag to not cache the image.

Workbench Explorer - A better way to browse drawers

Register To Post
	Top Previous Topic Next Topic

Currently Active Users Viewing This Thread: 1 ( 0 members and 1 Anonymous Users )