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

itcoin #1380

Merged
merged 44 commits into from
Mar 15, 2024
Merged

itcoin #1380

merged 44 commits into from
Mar 15, 2024

Conversation

wyatt-avilla
Copy link
Collaborator

@wyatt-avilla wyatt-avilla commented Mar 15, 2024

notes

  • changes in Ground_801C58E0 due to it returning it_802F2094
  • changes in struct Item due to the accesses in it_802F1F48
  • it_802F1630 and it_802F1CE0 seem to be non-matching for the same reason
  • it_802F1910 is so close to matching, it's just the condition within the first if statement

If any matching changes are obvious, please lmk and I'll make them. Otherwise, I'll mark this for review.

@wyatt-avilla wyatt-avilla marked this pull request as ready for review March 15, 2024 04:03
@ribbanya ribbanya merged commit 2cc16ed into doldecomp:master Mar 15, 2024
8 checks passed
@wyatt-avilla wyatt-avilla deleted the itcoin branch March 15, 2024 05:42
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