Skip to content

Conversation

@slaw-22
Copy link
Contributor

@slaw-22 slaw-22 commented Oct 12, 2025

I'd appreciate any early comments on this PR.

I'm not clear on the right names struct fields in scripting.h, but I might find these later! I'm also surprised that the global script var elements appear to be a different size to the local script var elements. It might be that the calling function LoadScriptVariable (not raw) might resolve this. Any thoughts on this, and any other comments would be appreciated.

@slaw-22 slaw-22 closed this Oct 12, 2025
@slaw-22 slaw-22 force-pushed the LoadScriptVariableRaw branch from 59d0ccd to b8dad60 Compare October 12, 2025 16:40
@slaw-22 slaw-22 reopened this Oct 12, 2025
@slaw-22
Copy link
Contributor Author

slaw-22 commented Oct 18, 2025

I'll put this PR out for approval now. My early work on LoadScriptVariableValue is likely to lead to an improvement in clarity for the structs in scripting.h. This will come in the next PR!

@slaw-22 slaw-22 marked this pull request as ready for review October 18, 2025 16:49
@AnonymousRandomPerson AnonymousRandomPerson merged commit 24c7a2e into pret:main Oct 18, 2025
2 checks passed
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.

2 participants