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

Initial support for BtVS: Chaos Bleeds (Xbox) #33

Merged
merged 3 commits into from
Apr 11, 2024

Conversation

joshbarrass
Copy link
Contributor

This adds the small special case changes needed to provide initial support for Buffy The Vampire Slayer: Chaos Bleeds. This has thus far been tested only against the Xbox version of the game.

Enables extraction of meshes and textures, allowing entities to be fully extracted, but currently does not support full extraction of maps (similar to Sphinx and the Shadow of Set Demo Disc and Sphinx and the Cursed Mummy). Spreadsheets are currently untested.

Reading the data size for Chaos Bleeds textures seemingly always returns a value of 88, which is obviously incorrect. The calculated size **must** be used instead. This adds an additional condition for Chaos Bleeds (`!(version == 170 && platform == Platform::Xbox)`) when deciding whether the `data_size` field should be included in `EXGeoTexture`. Disabling this field for Chaos Bleeds allows textures to be properly extracted.
Copy link
Member

@Swyter Swyter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes, quick stuff. Maybe it's worth looking into squashing the ReadMe changes into one, but that's just me nitpicking.

@Swyter Swyter requested a review from cohaereo April 9, 2024 13:33
@joshbarrass
Copy link
Contributor Author

Yeah I'm happy to squash the readme changes. I wasn't sure whether you'd just squash and merge anyway, so I left it as-is for the time being. I'll do that now.

@Swyter
Copy link
Member

Swyter commented Apr 11, 2024

@cohaereo seems busy, hope they don't mind me merging the few lines this entails. Thanks for squashing, Josh.

@Swyter Swyter merged commit 71dca6b into eurotools:main Apr 11, 2024
3 checks passed
@joshbarrass
Copy link
Contributor Author

No worries, thanks for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants