Skip to content

Conversation

@slaw-22
Copy link
Contributor

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

I have used script_var_type, script_var_def and special_episode_type from pmdsky debug. Please let me know if this is not Ok!

I've also used the relevant parts of script_var_id from pmdsky debug, but not all of it. I can bring the whole enum in if preferred.

VARTYPE_DUMMY_PADDING = 256
};

enum script_var_id {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might as well bring over the whole enum since it is known.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't see the rest of this enum yet. Did something go wrong with the commits?

@slaw-22 slaw-22 force-pushed the LoadScriptVariableValue branch from 742e220 to bb37701 Compare October 19, 2025 18:07
@AnonymousRandomPerson
Copy link
Collaborator

AnonymousRandomPerson commented Oct 24, 2025

I ran into a function that uses LoadScriptVariableValue, so I'll merge this as is and add the rest of the script_var_id enum in my next PR.

@AnonymousRandomPerson AnonymousRandomPerson merged commit 7464f4f into pret:main Oct 24, 2025
1 check passed
@slaw-22 slaw-22 deleted the LoadScriptVariableValue branch October 25, 2025 20:56
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