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

replace pointers with indexes in object_h #6105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Goober5000
Copy link
Contributor

@Goober5000 Goober5000 commented Apr 18, 2024

This refactors the object_h struct to store objects as indexes rather than pointers. This is a prerequisite for making the collection of objects dynamic in the future. See also #6100.

Follow-up to #5800 and #5829.

In draft pending tests. Tested and ready for review.

@Goober5000 Goober5000 added the refactor A cleanup/restructure of a feature for speed, simplicity, and/or maintainability label Apr 18, 2024
@Goober5000 Goober5000 added this to the Release 24.2 milestone Apr 18, 2024
This refactors the object_h struct to store objects as indexes rather than pointers.  This is a prerequisite for making the collection of objects dynamic in the future.  See also scp-fs2open#6100.

Follow-up to scp-fs2open#5800 and scp-fs2open#5829.
@Goober5000 Goober5000 marked this pull request as ready for review April 24, 2024 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor A cleanup/restructure of a feature for speed, simplicity, and/or maintainability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant