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

chore: Change entity and component logic to use bitstream references #1468

Merged
merged 3 commits into from
Feb 27, 2024

Conversation

jadebenn
Copy link
Collaborator

@jadebenn jadebenn commented Feb 25, 2024

Change to references from raw pointers.

Note: For your sanity, there are no unrelated changes in this or the other PRs. I specifically want to make that clear here given the sheer volume of files touched. The only changes included are those needed to make these pointers references. I have done my absolute best to keep these chunks as small as possible, but this one is huge despite my best efforts.

aronwk-aaron
aronwk-aaron previously approved these changes Feb 25, 2024
EmosewaMC
EmosewaMC previously approved these changes Feb 26, 2024
Copy link
Collaborator

@EmosewaMC EmosewaMC left a comment

Choose a reason for hiding this comment

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

github if you could please make the green just a bit more green that'd be great

same shpeel as the others, drop the test case performed here before merging.

@jadebenn
Copy link
Collaborator Author

Quest givers, vendors, combat, and general game play functioned normally over about an hour of game play following the main quest line from AG to NS to GF.

@jadebenn
Copy link
Collaborator Author

...though it looks like I will have to manually resolve some branch conflicts before I can merge later today

@jadebenn jadebenn merged commit b261e63 into main Feb 27, 2024
4 checks passed
@jadebenn jadebenn deleted the EntityCompBitStreamRefs branch February 27, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants