there is function destroy_single_object() in objects.c
(I'm talking about megapov 0.6a sources)
who can explain why it contains duplication of
destroying of bound and interior ?
there is such code (removed comments with NK):

  Destroy_Object(Object->Bound);
  Destroy_Interior((INTERIOR *)Object->Interior);
  Destroy_Transform(Object->UV_Trans);
  if (Object->Label!=NULL)
    POV_FREE(Object->Label);
  Destroy_Object (Object->Bound);
  Destroy_Interior((INTERIOR *)Object->Interior);

?

ABX

Post a reply to this message

Wlodzimierz ABX Skiba wrote:
> 
> there is function destroy_single_object() in objects.c
> (I'm talking about megapov 0.6a sources)
> who can explain why it contains duplication of
> destroying of bound and interior ?

	Yeah I also noticed these things in earlier version
of MegaPOV when I adapted part of its "persistent object"
code in my patch a few months ago. I suspect this is a mistake
in the code, which might produce weird bugs or might not if the
double-called functions take care enough of the pointers...
	The best would definitely to report the thing directly
to Nathan (I should have done this...). It's maybe one of the
strange things that make our Guru ;-) say the persistent object
handling is buggy !


*** Nicolas Calimet
*** http://pov4grasp.free.fr

Post a reply to this message