FreeEntity() crashes editor?

[Canardia]

I'm trying to spawn a cube when I place a model in editor, but Editor crashes when it tries to free the cube when I delete the model from Editor:

function class:CreateObject(model)
local object=self.super:CreateObject(model)
cube=CreateCube()
function object:Free()
	cube:Free()
end
end

[macklebee]

hmmm that code works fine for me... except if i drag and drop more two of the objects onto the screen, and then delete them it leaves the first cube, because the second cube create effectively is now 'cube'

 

you should parent the cube to the model in creation... then it gets deleted no matter what...

[Canardia]

Ok, I can parent it to the lua object, but not to the entity, since then the animation would not work anymore.

This seems to work fine, and it also deletes the cubes for each model you place in Editor:

require "scripts/class"
local class=CreateClass(...)
function class:CreateObject(model)
       local object=self.super:CreateObject(model)
       object.cube=CreateCube()
object.cube:SetPosition(object.model:GetPosition())
       function object:Free()
               self.cube:Free()
       end
end

[Rick]

The first way you were doing it was actually creating a global variable named cube. I'm guessing you don't want the cube to be global so making it apart of object would be what you want.

[Josh]

I usually like to have a lot of extra checks:

function class:CreateObject(model)

local object=self.super:CreateObject(model)
object.cube=CreateCube()

function object:Free()
	if self.cube~=nil then
			self.cube:Free()
			self.cube=nil
	end
	self.super:Free()
end

end

 

Don't forget self.super:Free()!

[Canardia]

Doesn't seem to be needed, as the model disappears when I press the delete key in Editor.

[Josh]

It may be invisible, but it will still be stored in memory and in the lua object table. The class will never get freed either.

[Canardia]

Ok, then it's better to free it also.