Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Heroes vanish when they die instead of showing their dead frames #765

Closed
ohrrpgce-bugbot opened this issue Oct 19, 2009 · 2 comments
Closed

Heroes vanish when they die instead of showing their dead frames #765

ohrrpgce-bugbot opened this issue Oct 19, 2009 · 2 comments

Comments

@ohrrpgce-bugbot
Copy link

@ohrrpgce-bugbot ohrrpgce-bugbot commented Oct 19, 2009

[bz#765]

When a hero dies in battle they simply vanish. Their death frame is not displayed. This is apparently a very recent bug. Perhaps caused by the new death fades for monsters?

From: @bob-the-hamster
Reported version: 20110506 Zenzizenzic

@ohrrpgce-bugbot
Copy link
Author

@ohrrpgce-bugbot ohrrpgce-bugbot commented Oct 19, 2009

Comment author: @bob-the-hamster

Yep, this started in svn revision 2951 (7361b8e), which is the one that added the new enemy dissolve types.

@ohrrpgce-bugbot
Copy link
Author

@ohrrpgce-bugbot ohrrpgce-bugbot commented Oct 19, 2009

Comment author: @bob-the-hamster

Aha! Got it.

When TMC implemented the new dissolve types, cleaned up the code so that bslot().dissolve was only set for enemies, which makes sense.

Unfortunately, .dissolve was also used by heroes to trigger the animate: GOSUB block to force the hero to use their death frame.

I fixed it so that hero's .dissolve gets set = 1 when they die, which fixes this bug.

This is not the most elegant way to handle hero death frames, but it is the way that worked before.

This area of code will get some further clean-up in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant