Clean up z_door_ana (#576)

* Clean up z_door_ana

* implement pr suggestions

* improve set type macro
This commit is contained in:
mzxrules 2022-03-24 20:38:13 -04:00 committed by GitHub
parent 17489d9a1d
commit fb3768c400
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 49 additions and 62 deletions

View File

@ -52,8 +52,7 @@ static ColliderCylinderInit sCylinderInit = {
{ 50, 10, 0, { 0, 0, 0 } },
};
// entrances grottos can link to statically, type DOORANA_TYPE_ADJACENT uses scene exit addresses instead
static u16 entrances[] = {
static u16 sEntranceIndexes[] = {
0x1A00, 0x1400, 0x1410, 0x1420, 0x1430, 0x1440, 0x1450, 0x1460,
0x1470, 0x1480, 0x1490, 0x14A0, 0x14B0, 0x14C0, 0x14D0,
};
@ -64,11 +63,12 @@ void DoorAna_SetupAction(DoorAna* this, DoorAnaActionFunc actionFunction) {
void DoorAna_Init(Actor* thisx, GlobalContext* globalCtx) {
DoorAna* this = THIS;
s32 grottoType = GET_DOORANA_TYPE(this);
s32 grottoType = DOORANA_GET_TYPE(&this->actor);
this->actor.shape.rot.y = this->actor.shape.rot.z = 0;
if ((grottoType == DOORANA_TYPE_UNK) || (grottoType == DOORANA_TYPE_HIDDEN)) {
if (grottoType == DOORANA_TYPE_HIDDEN) {
if (grottoType == DOORANA_TYPE_HIDDEN_STORMS || grottoType == DOORANA_TYPE_HIDDEN_BOMB) {
if (grottoType == DOORANA_TYPE_HIDDEN_BOMB) {
Collider_InitAndSetCylinder(globalCtx, &this->bombCollider, &this->actor, &sCylinderInit);
} else {
this->actor.flags |= ACTOR_FLAG_10; // always update
@ -86,27 +86,27 @@ void DoorAna_Init(Actor* thisx, GlobalContext* globalCtx) {
void DoorAna_Destroy(Actor* thisx, GlobalContext* globalCtx) {
DoorAna* this = THIS;
s32 grottoType = GET_DOORANA_TYPE(this);
s32 grottoType = DOORANA_GET_TYPE(&this->actor);
if (grottoType == DOORANA_TYPE_HIDDEN) {
if (grottoType == DOORANA_TYPE_HIDDEN_BOMB) {
Collider_DestroyCylinder(globalCtx, &this->bombCollider);
}
}
void DoorAna_WaitClosed(DoorAna* this, GlobalContext* globalCtx) {
s32 grottoIsOpen = 0;
u32 grottoType = GET_DOORANA_TYPE(this);
s32 grottoIsOpen = false;
u32 grottoType = DOORANA_GET_TYPE(&this->actor);
if (grottoType == DOORANA_TYPE_UNK) {
// in OOT decomp its marked as open with storms, but does not seem to open with storms in MM
if ((this->actor.xyzDistToPlayerSq < SQ(200.0f)) && (EnvFlags_Get(globalCtx, 5))) {
grottoIsOpen = 1;
if (grottoType == DOORANA_TYPE_HIDDEN_STORMS) {
//! @bug Implementation from OoT is not updated for MM, grotto does not open on Song of Storms
if (this->actor.xyzDistToPlayerSq < SQ(200.0f) && EnvFlags_Get(globalCtx, 5)) {
grottoIsOpen = true;
this->actor.flags &= ~ACTOR_FLAG_10; // always update OFF
}
} else {
if (this->bombCollider.base.acFlags & AC_HIT) { // bomb collision
grottoIsOpen = 1;
grottoIsOpen = true;
Collider_DestroyCylinder(globalCtx, &this->bombCollider);
} else {
@ -116,7 +116,7 @@ void DoorAna_WaitClosed(DoorAna* this, GlobalContext* globalCtx) {
}
if (grottoIsOpen) {
this->actor.params &= ~DOORANA_TYPE_BITRANGE;
DOORANA_SET_TYPE(&this->actor, DOORANA_TYPE_VISIBLE);
DoorAna_SetupAction(this, DoorAna_WaitOpen);
play_sound(NA_SE_SY_CORRECT_CHIME);
}
@ -126,56 +126,44 @@ void DoorAna_WaitClosed(DoorAna* this, GlobalContext* globalCtx) {
void DoorAna_WaitOpen(DoorAna* this, GlobalContext* globalCtx) {
Player* player = GET_PLAYER(globalCtx);
s32 dooranaType = GET_DOORANA_TYPE(this);
s8 pad[4];
s32 entranceIndex;
f32 playerHeightRel;
s32 grottoType = DOORANA_GET_TYPE(&this->actor);
if (Math_StepToF(&this->actor.scale.x, 0.01f, 0.001f) != 0) {
if ((this->actor.targetMode != 0) && (globalCtx->sceneLoadFlag == 0) && (globalCtx->unk_18B4A == 0) &&
(player->stateFlags1 & 0x80000000) && (player->unk_AE7 == 0)) {
if (Math_StepToF(&this->actor.scale.x, 0.01f, 0.001f)) {
if (this->actor.targetMode != 0 && globalCtx->sceneLoadFlag == 0 && globalCtx->unk_18B4A == 0 &&
(player->stateFlags1 & 0x80000000) && player->unk_AE7 == 0) {
if (dooranaType == DOORANA_TYPE_ADJACENT) {
s32 requiredScopeTemp;
if (grottoType == DOORANA_TYPE_VISIBLE_SCENE_EXIT) {
s32 exitIndex = DOORANA_GET_EXIT_INDEX(&this->actor);
// 300 uses scene exit addresses, not static DoorAna entrance addresses,
// eg. deku playground gets address in the NCT scene exit table
entranceIndex = GET_DOORANA_ADJACENT_ENTRANCE(this);
globalCtx->nextEntranceIndex = globalCtx->setupExitList[entranceIndex];
globalCtx->nextEntranceIndex = globalCtx->setupExitList[exitIndex];
} else {
// unused in vanilla, the highest params bits can directly index an address
entranceIndex = GET_DOORANA_DIRECT_ENTRANCE(this);
s32 destinationIdx = DOORANA_GET_ENTRANCE(&this->actor);
Play_SetupRespawnPoint(&globalCtx->state, 3, 0x4FF);
gSaveContext.respawn[3].pos.y = this->actor.world.pos.y;
gSaveContext.respawn[3].yaw = this->actor.home.rot.y;
// save the params lower byte for En_Torch to decide what item to use in the grotto chest
gSaveContext.respawn[3].data = GET_DOORANA_ITEMFLAGS(this);
// Stores item and chest flag that ACTOR_EN_TORCH uses for spawning the grotto chest
gSaveContext.respawn[3].data = DOORANA_GET_ITEMFLAGS(&this->actor);
// most grottos in the game use their zrotation as their entrance index, not params
if (DOORANA_TYPE_ROTATION_ENTRANCE(entranceIndex)) {
entranceIndex = GET_DOORANA_ROTATION_ENTRANCE(this);
if (destinationIdx < 0) {
destinationIdx = DOORANA_GET_EX_ENTRANCE(&this->actor);
}
globalCtx->nextEntranceIndex = entrances[entranceIndex];
globalCtx->nextEntranceIndex = sEntranceIndexes[destinationIdx];
}
DoorAna_SetupAction(this, DoorAna_GrabLink);
} else {
if ((func_801690CC(globalCtx) == 0) && ((player->stateFlags1 & 0x08800000) == 0) &&
(this->actor.xzDistToPlayer <= 20.0f) &&
(playerHeightRel = this->actor.playerHeightRel, (playerHeightRel >= -50.0f)) &&
(playerHeightRel <= 15.0f)) {
player->stateFlags1 |= 0x80000000;
this->actor.targetMode = 1;
} else if (func_801690CC(globalCtx) == 0 && (player->stateFlags1 & 0x08800000) == 0 &&
this->actor.xzDistToPlayer <= 20.0f && this->actor.playerHeightRel >= -50.0f &&
this->actor.playerHeightRel <= 15.0f) {
player->stateFlags1 |= 0x80000000;
this->actor.targetMode = 1;
} else {
this->actor.targetMode = 0;
}
} else {
this->actor.targetMode = 0;
}
}
@ -194,10 +182,10 @@ void DoorAna_GrabLink(DoorAna* this, GlobalContext* globalCtx) {
}
}
if ((this->actor.playerHeightRel <= 0.0f) && (this->actor.xzDistToPlayer > 20.0f)) {
if (this->actor.playerHeightRel <= 0.0f && this->actor.xzDistToPlayer > 20.0f) {
player = GET_PLAYER(globalCtx);
player->actor.world.pos.x = (Math_SinS(this->actor.yawTowardsPlayer) * 20.0f) + this->actor.world.pos.x;
player->actor.world.pos.z = (Math_CosS(this->actor.yawTowardsPlayer) * 20.0f) + this->actor.world.pos.z;
player->actor.world.pos.x = Math_SinS(this->actor.yawTowardsPlayer) * 20.0f + this->actor.world.pos.x;
player->actor.world.pos.z = Math_CosS(this->actor.yawTowardsPlayer) * 20.0f + this->actor.world.pos.z;
}
}

View File

@ -15,18 +15,17 @@ typedef struct DoorAna {
extern const ActorInit Door_Ana_InitVars;
#define DOORANA_TYPE_REGULAR 0x000
#define DOORANA_TYPE_UNK 0x100
#define DOORANA_TYPE_HIDDEN 0x200
#define DOORANA_TYPE_ADJACENT 0x300
#define DOORANA_TYPE_BITRANGE 0x300
#define DOORANA_TYPE_BITMASK 0x300
#define DOORANA_TYPE_VISIBLE 0x000
#define DOORANA_TYPE_HIDDEN_STORMS 0x100 // Incomplete, leftover from OoT
#define DOORANA_TYPE_HIDDEN_BOMB 0x200
#define DOORANA_TYPE_VISIBLE_SCENE_EXIT 0x300 // Destination dependent on scene exit list
#define GET_DOORANA_TYPE(this)(this->actor.params & 0x300)
#define DOORANA_TYPE_ROTATION_ENTRANCE(directEntrance)(directEntrance < 0)
#define GET_DOORANA_DIRECT_ENTRANCE(this) ((((this)->actor.params >> 0xC) & 0x7) - 1)
#define GET_DOORANA_ADJACENT_ENTRANCE(this) ((this)->actor.params & 0x1F)
#define GET_DOORANA_ROTATION_ENTRANCE(this) (this->actor.home.rot.z + 1)
#define GET_DOORANA_ITEMFLAGS(this) ((this)->actor.params & 0xFF)
#define DOORANA_GET_TYPE(thisx) ((thisx)->params & DOORANA_TYPE_BITMASK)
#define DOORANA_GET_ENTRANCE(thisx) ((((thisx)->params >> 12) & 0x7) - 1)
#define DOORANA_GET_EX_ENTRANCE(thisx) ((thisx)->home.rot.z + 1)
#define DOORANA_GET_EXIT_INDEX(thisx) ((thisx)->params & 0x1F)
#define DOORANA_GET_ITEMFLAGS(thisx) ((thisx)->params & 0xFF)
#define DOORANA_SET_TYPE(thisx, t) ((thisx)->params = ((thisx)->params & ~DOORANA_TYPE_BITMASK) + (t))
#endif // Z_DOOR_ANA_H