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: read vertex buffer size from level #1400

Merged
merged 4 commits into from
Jul 3, 2024

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Jul 2, 2024

Resolves #1398.

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

This replaces the fixed vertex buffer size of 1500 with a maximum value read from the level - so the "biggest" room, object or static mesh will determine the array size. In most cases it will be the biggest room, but this covers all bases. An alternative to doing the object/static check would be setting an initial value of 1500 and just iterating over the room meshes, so let me know if that's preferable.

This replaces the fixed vertex buffer size of 1500 with a maximum value
read from the level - so the "biggest" room, object or static mesh
determines the array size.

Resolves LostArtefacts#1398.
@lahm86 lahm86 added this to the 4.2 milestone Jul 2, 2024
@lahm86 lahm86 self-assigned this Jul 2, 2024
@lahm86 lahm86 requested review from rr- and walkawayy July 2, 2024 20:51
src/game/level.c Show resolved Hide resolved
src/game/output.h Outdated Show resolved Hide resolved
@lahm86 lahm86 requested a review from rr- July 2, 2024 21:20
Copy link
Collaborator

@walkawayy walkawayy left a comment

Choose a reason for hiding this comment

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

LGTM. Might be worth a README mention when talking about raised limits?

@rr- rr- force-pushed the issue-1398-vertex-buffer branch from 0433e8d to 2ab2e16 Compare July 3, 2024 07:41
@lahm86 lahm86 merged commit d0ba0e5 into LostArtefacts:develop Jul 3, 2024
1 check passed
@lahm86 lahm86 deleted the issue-1398-vertex-buffer branch July 3, 2024 08:36
@rr- rr- added the TR1 label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TR1 TRX bug A bug with TRX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TR1X bug: game crashes with large rooms
3 participants