memory leaks / what does this mean now

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 04:17, xxxxxxxx wrote:

This means you have created several objects in your code and forgot to free them.

Every time you create something e.g. with gNew() or you allocate an object like you do with BaseObjects BaseObject::Alloc(type); you have to free them after you're done using them.
You to this either with gDelete() or the class' Free method, like BaseObject::Free(op); Which of those it is, depends on how you created the object.

The only exception are objects whose ownership is taken over by Cinema (In that case the SDK sais "CINEMA 4D takes over the ownership" or "CINEMA 4D owns the pointed object").

Leaks are generally bad, because your plugin keeps on allocating more and more memory, without freeing the unused old memory blocks. If it's only a few bytes, no matter how long the plugin was in use, that's not too bad (but should be fixed anyway). But if you notice the amount of memory leaks is different every time, or the leaks get bigger if the plugin is in use for a longer time, you're in trouble. If somebody lets your plugin run for a couple of hours, it may crash because all the RAM is stuffed with rubbish.

If you manage to close all your leaks (or *at least* the ones that get bigger while the plugin is in use), you can regard your plugin as stable

Greetings,
Jack

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 07:35, xxxxxxxx wrote:

thanks.. now i'll look what could be the cause. at least i've hunted it down to one of three objectplugins
cheers,
ello

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 11:00, xxxxxxxx wrote:

So you're Ello, too?
How many accounts do you have?

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 11:10, xxxxxxxx wrote:

3 and thats just because the password email doesnt work.. i already sent a mail to maxon but didnt got a response so far.. i'd be glad to use my original "ello" account again

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 14:22, xxxxxxxx wrote:

i found that this line is responsible:

> `

  
\>  op = (BaseObject* )(profile->GetClone(0, NULL));  
\>

`

but using Free(op); doesnt help...

profile is created here:
> `

  
\>  BaseObject *profile = (BaseObject* )(bc- >GetLink(PROFILE,op->GetDocument()));  
\>

`

any further hints?

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 14:47, xxxxxxxx wrote:

BaseObject* profile = bc->GetObjectLink(PROFILE, op->GetDocument());
if (!profile) // do something

Always check for NULL in cases like this (the user could clear the LINK)!

What are you doing with the clone?

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 15:10, xxxxxxxx wrote:

the full part already looks like this:

> `

  
\>  BaseObject *op = BaseObject::Alloc(Osplinecircle);  
\>  if (profile)   
\>  {  
\>  profile ->SetEditorMode(MODE_OFF);  
\>  profile->SetRenderMode(MODE_OFF);  
\>  op = (BaseObject* )(profile->GetClone(0, NULL));  
\>  }  
\>  profile->InsertUnderLast(sweep);  
\>

`

if i dont uses the LINK the leaks dont appear

btw. this all happens in a for-loop

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 15:42, xxxxxxxx wrote:

You should move profile->InsertUnderLast(sweep) into the if{} block just in case profile is NULL, otherwise the code may crash.

You allocate to op instead of profile. I hope that is just a typing error here or is profile coming from someplace else?

You may need to create an AliasTrans and see if that removes the leaks from GetClone(). Normally you need one if the clone is going into another document than the one inwhich the original resides.

Finally, are you calling BaseObject::Free(op) in the for loop? Otherwise you would only be freeing the last iteration.

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 21:52, xxxxxxxx wrote:

yes, that was a typo. its "op" which is inserted..

i'll check that with the AliasTrans.. the manual sayi it can be NULL.. but i'll see what the manual offers about that..

I used the Free(op); inside the loop but that doesnt help

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 11/05/2009 at 22:12, xxxxxxxx wrote:

Okay.

AliasTrans can be NULL, but in the case of cloning into other documents it should not. The reason is that AliasTrans does the 'reassigning' in such cases (such as when the object has a link which isn't cloned and doesn't exist in the new document, it will clear it accordingly, and so on).

Still, I'm not sure if the GetClone() is the only cause of the leaks. Is it definitive or circumstantial?

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 12/05/2009 at 02:48, xxxxxxxx wrote:

just to clarify, i am not cloning it into another document.

the leaking appears as soon as i use this line:
op = (BaseObject* )(profile->GetClone(0, NULL));

when i comment it out, everything is fine

btw. it only happens when i have dragged something into the link field and the condition "if(profile)" is true

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 12/05/2009 at 04:45, xxxxxxxx wrote:

definately has something to do with the combination of cloning the linked object since using a child object (op->GetDown(); does work with this line:

> `

  
\>  BaseObject *clone = static_cast<BaseObject*>(child->GetClone(0, NULL));  
\>

`

any ideas?

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 12/05/2009 at 08:35, xxxxxxxx wrote:

Instead of:
BaseObject *op = BaseObject::Alloc(Osplinecircle);

do:
Baseobject* op = NULL;

You don´t need to allocate op if you fill it with a clone later on. C4D does the memory allocation for the clone internally when you call GetClone().

Helper

THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

On 12/05/2009 at 08:42, xxxxxxxx wrote:

thanks! that was my idea to have at least a splinecircle when the user didnt drag a spline to the link field.

but just a second ago i found out that somehow exactly that creation of the circle beforehand was the issue. now there is no leaking and i am just feeling lighter now

thanks for all your help here!

this is how i do it now:

> `

  
\>    
\>                 if(profileLink)       
\>                 {  
\>                      BaseObject *profile = static_cast<BaseObject*>(profileLink->GetClone(0, NULL));  
\>                      profile->InsertUnderLast(sweep);  
\>                 }  
\>

`