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

Refactor room-related collections #617

Merged
merged 8 commits into from
Apr 15, 2024

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Apr 15, 2024

Part of #468.

Checklist

Description

Further work on converting arrays to lists, mainly things related to rooms this time. The sub structures are still quite array-heavy, like TRRoom but we will get to those separately as they're also heavily referenced elsewhere.

The next - and hopefully final - stage for the top level variables will be meshes, the remaining texture data and sound effects and then we can look at data re-organisation.

lahm86 added 8 commits April 15, 2024 11:11
All levels store a uint before the room count which is always 0 and is ignored in all games during read, It is believed to be a level number field. We store this for 1-3 in case it becomes usable, TR4/5 is a little more awkward because of where it's kept in the compressed part of the file, so for now we just keep it in place.
TRGE updates room flags when the sunset is enabled.
@lahm86 lahm86 added this to the 1.9.0 milestone Apr 15, 2024
@lahm86 lahm86 self-assigned this Apr 15, 2024
@lahm86 lahm86 requested review from rr-, chreden and makotocchi April 15, 2024 18:29
@lahm86 lahm86 merged commit 4e109d6 into LostArtefacts:master Apr 15, 2024
2 checks passed
@lahm86 lahm86 deleted the issue-468-level-refactoring branch April 15, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants