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

[LEAK INFORMED] Documented the unk19 item field #1575

Merged
merged 2 commits into from
Dec 18, 2021

Conversation

LOuroboros
Copy link
Contributor

Description

This field is called "convenience button" in the original source code of Pokémon Emerald.
My personal guess is that the field was meant to signal that the Player should be allowed to register the items that have this field set to 1/TRUE.

As all the items from the no. 360 (FRLG's Bicycle) to the no. 366 (Teachy TV) have the flag set to "1", I suspect that the items inserted afterward, which a Player has no reason to register for usage with the Select button with the exception of the Powder Jar, simply have it set to 1/TRUE by mistake, either because they were automatically generated or because whoever was in charge of writing the data of those items simply followed the lead of the items that come before.

Fun story:
In the original code, there's actually 2 fields that share the same name.
There's this field in the Items' data that Pret currently calls unk19, and then there's the field that Pret calls "registeredItem" inside of the SaveBlock1 struct.
Both of them have the exact same name in the original source code.
That coincidence is the sole reason why I was able to come up with this theory of mine.

Discord contact info

Lunos#4026

lemondrop81 pushed a commit to lemondrop81/pokeemerald that referenced this pull request Dec 18, 2021
include/item.h Outdated Show resolved Hide resolved
@GriffinRichards GriffinRichards merged commit 32a9ac6 into pret:master Dec 18, 2021
github-actions bot pushed a commit that referenced this pull request Dec 18, 2021
@LOuroboros LOuroboros deleted the benriBotan branch December 18, 2021 16:51
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