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

d_a_item -> Layout and Enum #92

Merged
merged 15 commits into from
Nov 9, 2024

Conversation

elijah-thomas774
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@CovenEsme CovenEsme left a comment

Choose a reason for hiding this comment

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

Updated some item names from HDR :p

include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
include/d/a/d_a_itembase.h Outdated Show resolved Hide resolved
Comment on lines 208 to 209
/* 0x18D */ ITEM_LIFE_MEDAL_AGAIN = 397,
/* 0x18E */ ITEM_POUCH_EXPANSION_GODDESS_CHEST = 398,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are these ones actually defined? Seems weird that they'd have 0 -> 200, 397, 398, 511

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just copied what was from ghidra, i figured it couldn't hurt to have here. Ill put a comment in

src/toBeSorted/special_item_drop_mgr.cpp Outdated Show resolved Hide resolved
@elijah-thomas774 elijah-thomas774 marked this pull request as ready for review November 7, 2024 17:16
@elijah-thomas774
Copy link
Collaborator Author

I dont think i currently have the motivation to tackle this one xD

@elijah-thomas774 elijah-thomas774 changed the title d_a_item d_a_item -> Layout and Enum Nov 7, 2024
src/REL/d/t/d_t_tumble_weed.cpp Outdated Show resolved Hide resolved
Copy link
Collaborator

@robojumper robojumper left a comment

Choose a reason for hiding this comment

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

Aww, I was hoping we'd see dAcItem_c::init matched 😄

Copy link
Collaborator

@robojumper robojumper left a comment

Choose a reason for hiding this comment

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

wrong button

@elijah-thomas774 elijah-thomas774 merged commit 126e1a9 into zeldaret:main Nov 9, 2024
1 check passed
@elijah-thomas774 elijah-thomas774 deleted the d_a_item branch November 13, 2024 03:45
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.

3 participants