Conversation
32ed6eb to
76d0bb0
Compare
| Player* player = GET_PLAYER(gPlayState); | ||
| Player* player = va_arg(args, Player*); |
There was a problem hiding this comment.
Does this make a difference?
There was a problem hiding this comment.
yes. this is what caused the bug in the first place. GET_PLAYER always gets link
There was a problem hiding this comment.
to elaborate on this:
we make it in here as both link and dark link. the previous logic (before moving this to shipinit) had both link and dark link running both the actionfunc and usetunicboots. by using GET_PLAYER, we were running both the actionfunc and usetunicboots as link for both link and dark link.
so we need to know if we're in here because of link or dark link
i could switch to passing in the actor id instead and then using GET_PLAYER if you'd prefer that
There was a problem hiding this comment.
Nope that's fine, I just misunderstood what was happening.
There was a problem hiding this comment.
Yeah we had to do something similar in a 2ship hook because of player common being used by Kafei and Link.
fixes #5016
Build Artifacts