CanCrash bug

CanCrash bug

Discuss the source code and development of Spring Engine in general from a technical point of view. Patches go here too.

Moderator: Moderators

Post Reply
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

CanCrash bug

Post by Argh »

CanCrash re-introduced the bug into Spring, where death-animations weren't allowed to operate during the process of removing the Unit, which effected anything using CAirMoveType. Setting it to 1 or 0 did not change this, in testing.
Warlord Zsinj
Imperial Winter Developer
Posts: 3742
Joined: 24 Aug 2004, 08:59

Re: CanCrash bug

Post by Warlord Zsinj »

Just set spring to CanCrash=0; that should solve the problem.

[/bad joke]

:lol:
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: CanCrash bug

Post by Argh »

LOL!

If only it were that simple. I would have to keep it on the experimental "CanCrash" mode, though, just to be different.
User avatar
KDR_11k
Game Developer
Posts: 8293
Joined: 25 Jun 2006, 08:44

Re: CanCrash bug

Post by KDR_11k »

canCrash only changed the if statement that checks if the plane should enter crashing mode or just explode. If the death anim doesn't play it wouldn't have played without the change either.
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: CanCrash bug

Post by Argh »

It played after ILMTitan's patch... maybe I have that confused with Lurker's bit.

Either way, meh, I'm back to having all of the death-animation code for flying things broken again, anything that wants to execute past the frame of the Unit's death does not execute...
User avatar
yuritch
Spring 1944 Developer
Posts: 1018
Joined: 11 Oct 2005, 07:18

Re: CanCrash bug

Post by yuritch »

Was that patch even added to svn? AFAIR all we had was a custom compiled binary, it wasn't in the trunk.
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: CanCrash bug

Post by Argh »

Ah, that must be it, it never got committed. I thought it was working after ILMTitan's patch, but I must be wrong...
User avatar
yuritch
Spring 1944 Developer
Posts: 1018
Joined: 11 Oct 2005, 07:18

Re: CanCrash bug

Post by yuritch »

I just re-read that thread, and it never mentions the patch being added to svn. Someone should add it probably (but it still had some strange things left, like 'dying' air units stuck in place until Killed() returns, which may or may not be intentional).
User avatar
ILMTitan
Spring Developer
Posts: 410
Joined: 13 Nov 2004, 08:35

Re: CanCrash bug

Post by ILMTitan »

The changes I have made to AirMoveType so far should not have changed any functionality. It was just refactoring so I could wrap my head around what it was doing, in preparation to write an entirely new air movement class (which is what I am working on now). If it appears I broke anything, please let me know.
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: CanCrash bug

Post by Argh »

No, it was my mistake, I was confused about what did what when, been a little busy and lost track of what's officially fixed :P
User avatar
lurker
Posts: 3842
Joined: 08 Jan 2007, 06:13

Re: CanCrash bug

Post by lurker »

Yeah, yeah, I'll wrap it up and we can get it in svn.
Post Reply

Return to “Engine”