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

output: use common texture animation routine #2402

Merged
merged 5 commits into from
Jan 28, 2025

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Jan 27, 2025

Resolves #2401.

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
  • I have added a readme entry about my new feature or OG bug fix, or it is a different change

Description

The main goal here was to get a common texture animation function, so adding support for animated sprites in TR2 was a given. Test level below for checking. I've also moved the range storage to the common output module.

I also realised I'd mistakenly logged the photo mode flip map fix in the TR2 changelog so have removed that (photo mode itself is unreleased).

WALL.zip

This uses the same logic as TR1 to allow sprites to be animated.

Resolves LostArtefacts#2401.
Photo mode is unreleased in TR2 so the entry for LostArtefacts#2362 is not relevant.
Moves the anim texture initialise function to match the header.
This moves the specifics of cycling through animated textures to TRX.
This removes the global animated texture array and replaces with an
accessor.
@lahm86 lahm86 added Feature New functionality Internal The invisible stuff TR2 TR1 labels Jan 27, 2025
@lahm86 lahm86 self-assigned this Jan 27, 2025
@lahm86 lahm86 requested review from a team as code owners January 27, 2025 21:40
@lahm86 lahm86 requested review from rr-, walkawayy and aredfan and removed request for a team January 27, 2025 21:40
@lahm86 lahm86 merged commit 40729e6 into LostArtefacts:develop Jan 28, 2025
8 checks passed
@lahm86 lahm86 deleted the further-anim-textures branch January 28, 2025 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New functionality Internal The invisible stuff TR1 TR2
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Feature: add support for animated sprites
3 participants