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

Decompile creature objects setup #1720

Closed
wants to merge 0 commits into from
Closed

Conversation

rr-
Copy link
Collaborator

@rr- rr- commented Oct 15, 2024

Checklist

  • I have read the coding conventions
  • I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

#1691

This decompiles routines for initializing all creatures and the armed skidoo. It may impact savegame functionality, potentially causing crashes when loading an original save, with issues possibly appearing individually in each level.

@rr- rr- added Internal The invisible stuff TR2 labels Oct 15, 2024
@rr- rr- self-assigned this Oct 15, 2024
@rr- rr- requested review from a team as code owners October 15, 2024 21:55
@rr- rr- requested review from lahm86, walkawayy and aredfan and removed request for a team October 15, 2024 21:55
@aredfan
Copy link
Collaborator

aredfan commented Oct 16, 2024

I found just this one Issue:

  • The game crashes when small spiders are spawned. (/play 1, tp 0, go through the locked door.)

@rr- rr- force-pushed the tr2-decompile-objects-setup branch from 92ed8d5 to f15b228 Compare October 16, 2024 18:10
@rr-
Copy link
Collaborator Author

rr- commented Oct 16, 2024

Thanks @lahm86 and @aredfan – fixed.

@rr- rr- closed this Oct 16, 2024
@rr- rr- force-pushed the tr2-decompile-objects-setup branch from f15b228 to d531d0b Compare October 16, 2024 18:31
@rr- rr- deleted the tr2-decompile-objects-setup branch October 16, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal The invisible stuff TR2
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants