Crash when activating collisions after cell change

Everything about development and the OpenMW source code.
jhooks1
Posts: 780
Joined: 06 Aug 2011, 21:34

Re: Crash when activating collisions after cell change

Post by jhooks1 »

I think the problem is with Player, and it's scenenode/handle being erased.
User avatar
Zini
Posts: 5538
Joined: 06 Aug 2011, 15:16

Re: Crash when activating collisions after cell change

Post by Zini »

Why would a change to scene nodes result in a crash deep in the physics system?
jhooks1
Posts: 780
Joined: 06 Aug 2011, 21:34

Re: Crash when activating collisions after cell change

Post by jhooks1 »

I think it is the physics handle that is being removed along with the npc/creature physics handles?
User avatar
Zini
Posts: 5538
Joined: 06 Aug 2011, 15:16

Re: Crash when activating collisions after cell change

Post by Zini »

There is no physics handle, but the OGRE handle is also used by the physics system to identify objects. Still doesn't explain what I see in the debugger.
jhooks1
Posts: 780
Joined: 06 Aug 2011, 21:34

Re: Crash when activating collisions after cell change

Post by jhooks1 »

I think the problem may lie in the implementation of unloadCell()
User avatar
gus
Posts: 390
Joined: 11 Aug 2011, 15:41

Re: Crash when activating collisions after cell change

Post by gus »

Could anyone try with the latest SVN version of bullet? That bug seems familiar to me, and it's seems that bullet is not doing some cleaning up well, leading to a crash.

Edit: never mind. werdanith tried it, it doesn't work.
User avatar
gus
Posts: 390
Joined: 11 Aug 2011, 15:41

Re: Crash when activating collisions after cell change

Post by gus »

I somehow fixed the crash... I removed for testing purpose a line that should be necessary, and the crash simply vanished, without breaking (or so it seems) the physic :p


Edit: changes are pushed in my master branch of openengine.
User avatar
Zini
Posts: 5538
Joined: 06 Aug 2011, 15:16

Re: Crash when activating collisions after cell change

Post by Zini »

This is somewhat unnerving. I guess if everything else is ready, we will have to release 0.12.0 with this fix, but removing a crash without fully understanding what went wrong is an invitation for further trouble.
jhooks1
Posts: 780
Joined: 06 Aug 2011, 21:34

Re: Crash when activating collisions after cell change

Post by jhooks1 »

Good Job 8-)
User avatar
Greendogo
Posts: 1467
Joined: 26 Aug 2011, 02:04

Re: Crash when activating collisions after cell change

Post by Greendogo »

How exactly can removing a necessary line and having it not crash be a solution? Doesn't it break something else? (I ask while not looking at the code for myself)
Post Reply