minor inline cleanup

This commit is contained in:
LagoLunatic
2024-03-11 18:12:15 -04:00
parent 44761305b9
commit 029ddf6bb9
19 changed files with 49 additions and 57 deletions
+3 -11
View File
@@ -16,9 +16,7 @@
fopAcM_OnCondition(ptr, fopAcCnd_INIT_e); \
}
class J3DModelData; // placeholder
class JKRHeap;
class cM3dGPla;
class J3DModelData;
class daItem_c;
struct fopAcM_prmBase_class {
@@ -79,10 +77,6 @@ class dKy_tevstr_c;
typedef int (*heapCallbackFunc)(fopAc_ac_c*);
typedef int (*createFunc)(void*);
// struct DOUBLE_POS {
// double x, y, z;
// };
inline s8 fopAcM_GetRoomNo(fopAc_ac_c* pActor) {
return pActor->current.roomNo;
}
@@ -99,7 +93,7 @@ inline MtxP fopAcM_GetMtx(fopAc_ac_c* pActor) {
return pActor->cullMtx;
}
inline bool fopAcM_checkStatus(fopAc_ac_c* pActor, u32 status) {
inline bool fopAcM_CheckStatus(fopAc_ac_c* pActor, u32 status) {
return pActor->actor_status & status;
}
@@ -108,7 +102,7 @@ inline u32 fopAcM_checkCarryNow(fopAc_ac_c* pActor) {
}
inline u32 fopAcM_checkHookCarryNow(fopAc_ac_c* pActor) {
return fopAcM_checkStatus(pActor, fopAcStts_HOOK_CARRY_e);
return fopAcM_CheckStatus(pActor, fopAcStts_HOOK_CARRY_e);
}
inline u32 fopAcM_GetParam(void* pActor) {
@@ -566,8 +560,6 @@ void fopAcM_cancelCarryNow(fopAc_ac_c* i_this);
s32 fopAcM_otoCheck(fopAc_ac_c*, f32);
BOOL fopAcM_viewCutoffCheck(fopAc_ac_c* actor, f32 param_2);
BOOL fopAcM_getGroundAngle(fopAc_ac_c* actor, csXyz* p_angle);
// void vectle_calc(DOUBLE_POS*, cXyz*);
// void get_vectle_calc(cXyz*, cXyz*, cXyz*);
static const char* fopAcM_getProcNameString(fopAc_ac_c* p_actor);
+7 -7
View File
@@ -15,13 +15,13 @@ typedef int (*FastCreateReqFunc)(void*);
typedef void (*fpcM_ManagementFunc)(void);
typedef int (*fpcM_DrawIteraterFunc)(void*, void*);
inline u32 fpcM_GetID(const void* pProc) {
inline u32 fpcM_GetID(void* pProc) {
return pProc != NULL ? ((base_process_class*)pProc)->mBsPcId : fpcM_ERROR_PROCESS_ID_e;
}
inline s16 fpcM_GetName(const void* pActor) {
inline s16 fpcM_GetName(void* pActor) {
return ((base_process_class*)pActor)->mProcName;
}
inline u32 fpcM_GetParam(const void* pActor) {
inline u32 fpcM_GetParam(void* pActor) {
return ((base_process_class*)pActor)->mParameters;
}
@@ -29,7 +29,7 @@ inline void fpcM_SetParam(void* p_actor, u32 param) {
((base_process_class*)p_actor)->mParameters = param;
}
inline s16 fpcM_GetProfName(const void* pActor) {
inline s16 fpcM_GetProfName(void* pActor) {
return ((base_process_class*)pActor)->mProfName;
}
@@ -38,7 +38,7 @@ inline int fpcM_Create(s16 procName, FastCreateReqFunc createFunc, void* process
process);
}
inline s16 fpcM_DrawPriority(const void* param_0) {
inline s16 fpcM_DrawPriority(void* param_0) {
return fpcLf_GetPriority((leafdraw_class*)param_0);
}
@@ -62,7 +62,7 @@ inline process_profile_definition* fpcM_GetProfile(void* proc) {
return (process_profile_definition*)((base_process_class*)proc)->mpProf;
}
inline void* fpcM_GetAppend(const void* proc) {
inline void* fpcM_GetAppend(void* proc) {
return ((base_process_class*)proc)->mpUserData;
}
@@ -74,7 +74,7 @@ inline void* fpcM_LyJudge(process_node_class* i_node, fpcLyIt_JudgeFunc i_func,
return fpcLyIt_Judge(&i_node->mLayer, i_func, i_data);
}
inline s8 fpcM_CreateResult(const void* pActor) {
inline s8 fpcM_CreateResult(void* pActor) {
return ((base_process_class*)pActor)->mCreateResult;
}
+2 -2
View File
@@ -14,8 +14,8 @@ struct mDoLib_clipper {
mClipper.calcViewFrustum();
}
static s32 clip(const Mtx m, const Vec* max, const Vec* min) {
return mClipper.clip(m, (Vec*)max, (Vec*)min);
static s32 clip(const Mtx m, Vec* max, Vec* min) {
return mClipper.clip(m, max, min);
}
static s32 clip(const Mtx m, Vec center, f32 radius) {
+2 -2
View File
@@ -228,7 +228,7 @@ BOOL enemy_ice(enemyice* ei) {
if (ei->m00C != 1) {
ac->attention_info.flags |= fopAc_Attn_ACTION_CARRY_e;
ac->attention_info.distances[4] = 0x12;
if (fopAcM_checkStatus(ac, fopAcStts_CARRY_e)) {
if (fopAcM_CheckStatus(ac, fopAcStts_CARRY_e)) {
ac->attention_info.flags &= ~fopAc_Attn_ACTION_CARRY_e;
ei->mState = 3;
if (ei->m00C == 2) {
@@ -239,7 +239,7 @@ BOOL enemy_ice(enemyice* ei) {
break;
case 3: // Frozen and being carried around by the player
frozen = TRUE;
if (!fopAcM_checkStatus(ac, fopAcStts_CARRY_e)) {
if (!fopAcM_CheckStatus(ac, fopAcStts_CARRY_e)) {
if (fopAcM_GetSpeedF(ac) > 0.0f) {
ei->mSpeedF = 25.0f + g_regHIO.mChild[0].mFloatRegs[5];
ei->mSpeed.y = 20.0f + g_regHIO.mChild[0].mFloatRegs[6];
+1 -1
View File
@@ -1466,7 +1466,7 @@ void daAgb_c::modeMove() {
if (attList) {
fopAc_ac_c* r3 = attList->getActor();
if (r3) {
if (fopAcM_CheckStatusMap(r3, 0) && !fopAcM_checkStatus(r3, fopAcStts_BOSS_e) && fopAcM_GetName(r3) != PROC_FGANON) {
if (fopAcM_CheckStatusMap(r3, 0) && !fopAcM_CheckStatus(r3, fopAcStts_BOSS_e) && fopAcM_GetName(r3) != PROC_FGANON) {
current.pos = r3->current.pos;
home.pos = r3->current.pos;
setTargetID(attList->getPid());
+6 -6
View File
@@ -127,7 +127,7 @@ static BOOL daAM2_Draw(am2_class* i_this) {
i_this->mpBrkAnm->remove(model->getModelData());
i_this->mpBtkAnm->remove(model->getModelData());
if (!fopAcM_checkStatus(i_this, fopAcStts_CARRY_e)) {
if (!fopAcM_CheckStatus(i_this, fopAcStts_CARRY_e)) {
dComIfGd_setSimpleShadow2(
&i_this->current.pos, i_this->mAcch.GetGroundH(), 50.0f, i_this->mAcch.m_gnd,
0, 1.0f, dDlst_shadowControl_c::getSimpleTex()
@@ -684,7 +684,7 @@ static void action_mahi(am2_class* i_this) {
fopAcM_seStart(actor, JA_SE_CM_AM2_JUMP, 0);
i_this->mCountUpTimers[1] = 0;
}
if (fopAcM_checkStatus(actor, fopAcStts_CARRY_e)) {
if (fopAcM_CheckStatus(actor, fopAcStts_CARRY_e)) {
i_this->mAcchRadius = 40.0f + g_regHIO.mChild[8].mFloatRegs[10];
i_this->mbMadeWaterSplash = false;
i_this->mRippleCb.end();
@@ -708,7 +708,7 @@ static void action_mahi(am2_class* i_this) {
if (i_this->mPickedUpYPos + 10.0f <= actor->current.pos.y) {
cLib_addCalcAngleS2(&actor->shape_angle.y, actor->current.angle.y, 1, 0x1000);
}
if (!fopAcM_checkStatus(actor, fopAcStts_CARRY_e)) {
if (!fopAcM_CheckStatus(actor, fopAcStts_CARRY_e)) {
i_this->mAcchRadius = 40.0f + g_regHIO.mChild[8].mFloatRegs[10];
i_this->mBodyCyl.OnCoSetBit();
if (actor->speedF > 0.0f) {
@@ -807,7 +807,7 @@ static void action_mahi(am2_class* i_this) {
i_this->mAction = ACTION_DOUSA;
i_this->mState = 0;
if (fopAcM_checkStatus(actor, fopAcStts_CARRY_e)) {
if (fopAcM_CheckStatus(actor, fopAcStts_CARRY_e)) {
fopAcM_cancelCarryNow(actor);
actor->attention_info.flags &= ~fopAc_Attn_ACTION_CARRY_e;
actor->gravity = -4.0f;
@@ -827,7 +827,7 @@ static void action_mahi(am2_class* i_this) {
}
if (i_this->mCountDownTimers[3] == 1) {
if (fopAcM_checkStatus(actor, fopAcStts_CARRY_e)) {
if (fopAcM_CheckStatus(actor, fopAcStts_CARRY_e)) {
fopAcM_cancelCarryNow(actor);
actor->attention_info.flags &= ~fopAc_Attn_ACTION_CARRY_e;
}
@@ -840,7 +840,7 @@ static void action_mahi(am2_class* i_this) {
}
}
if (fopAcM_checkStatus(actor, fopAcStts_CARRY_e) || i_this->mState == 0xF || !week_atari_check(i_this)) {
if (fopAcM_CheckStatus(actor, fopAcStts_CARRY_e) || i_this->mState == 0xF || !week_atari_check(i_this)) {
body_atari_check(i_this);
}
}
+2 -2
View File
@@ -193,7 +193,7 @@ void daHookshot_rockLineCallback(fopAc_ac_c* hookshot_actor, dCcD_GObjInf* objIn
f32 f1 = i_this->m2CC.abs2(i_this->current.pos);
if (f1 > f31) {
i_this->m2CC = *objInf->GetAtHitPosP();
if (fopAcM_checkStatus(collided_actor, fopAcStts_UNK80000_e | fopAcStts_UNK200000_e | fopAcStts_UNK10000000_e)) {
if (fopAcM_CheckStatus(collided_actor, fopAcStts_UNK80000_e | fopAcStts_UNK200000_e | fopAcStts_UNK10000000_e)) {
i_this->m2B0 = 1;
} else {
i_this->m2B0 = 0;
@@ -319,7 +319,7 @@ static BOOL daHookshot_IsDelete(daHookshot_c* i_this) {
BOOL daHookshot_c::hookshot_delete() {
if (mCarryActorID != fpcM_ERROR_PROCESS_ID_e) {
fopAc_ac_c* hooked_actor = fopAcM_SearchByID(mCarryActorID);
if (hooked_actor && fopAcM_checkStatus(hooked_actor, fopAcStts_HOOK_CARRY_e)) {
if (hooked_actor && fopAcM_CheckStatus(hooked_actor, fopAcStts_HOOK_CARRY_e)) {
fopAcM_cancelHookCarryNow(hooked_actor);
mCarryActorID = fpcM_ERROR_PROCESS_ID_e;
}
+1 -1
View File
@@ -798,7 +798,7 @@ BOOL daItem_c::checkItemDisappear() {
if (mItemStatus == STATUS_UNK4) {
disappearing = FALSE;
}
if (checkFlag(FLAG_BOOMERANG) || checkFlag(FLAG_HOOK) || fopAcM_checkStatus(this, fopAcStts_HOOK_CARRY_e)) {
if (checkFlag(FLAG_BOOMERANG) || checkFlag(FLAG_HOOK) || fopAcM_CheckStatus(this, fopAcStts_HOOK_CARRY_e)) {
disappearing = FALSE;
show();
}
+2 -2
View File
@@ -1136,7 +1136,7 @@ BOOL daNpc_Md_c::lightHitCheck() {
if (mCps.ChkAtHit()) {
fopAc_ac_c* hitActor = mCps.GetAtHitAc();
if (fopAcM_checkStatus(this, fopAcStts_CARRY_e) && !isNoCarryAction()) {
if (fopAcM_CheckStatus(this, fopAcStts_CARRY_e) && !isNoCarryAction()) {
if (hitActor != dComIfGp_getLinkPlayer() && m3058.getEmitter() == NULL) {
dComIfGp_particle_set(0x8232, &current.pos, NULL, NULL, 0xFF, &m3058);
}
@@ -2964,7 +2964,7 @@ BOOL daNpc_Md_c::draw() {
dComIfGd_getXluList()->entryImm(&m0B70, 0x1F);
}
if (!isShipRide() && !fopAcM_checkStatus(this, fopAcStts_CARRY_e) && cLib_checkBit(m30F0, 0x20000UL)) {
if (!isShipRide() && !fopAcM_CheckStatus(this, fopAcStts_CARRY_e) && cLib_checkBit(m30F0, 0x20000UL)) {
cXyz shadowPos(current.pos.x, current.pos.y + 150.0f, current.pos.z);
mShadowId = dComIfGd_setShadow(
mShadowId, 0, mpMorf->getModel(), &shadowPos, 800.0f, 20.0f,
+1 -1
View File
@@ -174,7 +174,7 @@ BOOL daPy_npc_c::checkNowPosMove(const char* pName) {
return TRUE;
}
return fopAcM_checkStatus(this, fopAcStts_UNK800_e);
return fopAcM_CheckStatus(this, fopAcStts_UNK800_e);
}
/* 8015AC74-8015AD20 .text drawDamageFog__10daPy_npc_cFv */
+1 -1
View File
@@ -565,7 +565,7 @@ fopAc_ac_c* dAttList_c::getActor() {
/* 8009FA98-8009FAB4 .text setActor__10dAttList_cFP10fopAc_ac_c */
void dAttList_c::setActor(fopAc_ac_c* i_actor) {
mActorID = fpcM_GetID(i_actor);
mActorID = fopAcM_GetID(i_actor);
}
/* 8009FAB4-8009FACC .text getPId__10dAttHint_cFPv */
+2 -2
View File
@@ -65,7 +65,7 @@ void dCcD_GStts::Move() {
void dCcD_Stts::Init(int weight, int param_1, fopAc_ac_c* pActor) {
u32 procId;
if (pActor) {
procId = fpcM_GetID(pActor);
procId = fopAcM_GetID(pActor);
} else {
procId = fpcM_ERROR_PROCESS_ID_e;
}
@@ -73,7 +73,7 @@ void dCcD_Stts::Init(int weight, int param_1, fopAc_ac_c* pActor) {
s32 roomNo;
if (pActor) {
roomNo = pActor->current.roomNo;
roomNo = fopAcM_GetRoomNo(pActor);
} else {
roomNo = 0;
OnNoActor();
+1 -1
View File
@@ -202,7 +202,7 @@ fopAc_ac_c* cc_at_check(fopAc_ac_c* tgActor, CcAtInfo* atInfo) {
atInfo->m0C.y = cM_atan2s(-f2, -f0);
if (atInfo->mpObj->ChkAtType(AT_TYPE_HOOKSHOT)) {
if (fopAcM_checkStatus(tgActor, fopAcStts_UNK80000_e | fopAcStts_HOOK_CARRY_e | fopAcStts_UNK200000_e)) {
if (fopAcM_CheckStatus(tgActor, fopAcStts_UNK80000_e | fopAcStts_HOOK_CARRY_e | fopAcStts_UNK200000_e)) {
atInfo->mDamage = 0;
}
}
+1 -1
View File
@@ -147,7 +147,7 @@ u8 dDoor_info_c::drawCheck(int mode) {
/* 8006B8AC-8006B954 .text checkExecute__12dDoor_info_cFv */
u8 dDoor_info_c::checkExecute() {
mFrontCheck = frontCheck();
if (fopAcM_checkStatus(this, fopAcStts_UNK1000_e))
if (fopAcM_CheckStatus(this, fopAcStts_UNK1000_e))
return 1;
if (eventInfo.checkCommandDemoAccrpt() || eventInfo.checkCommandDoor())
+7 -7
View File
@@ -632,22 +632,22 @@ s32 dEvt_control_c::moveApproval(void* actor) {
break;
}
if (fopAcM_checkStatus(i_ac, fopAcStts_FORCEMOVE_e))
if (fopAcM_CheckStatus(i_ac, fopAcStts_FORCEMOVE_e))
return dEvtMove_FORCE_e;
if (getMode() == dEvtMode_TALK_e && fopAcM_checkStatus(i_ac, fopAcStts_UNK40_e))
if (getMode() == dEvtMode_TALK_e && fopAcM_CheckStatus(i_ac, fopAcStts_UNK40_e))
return dEvtMove_MOVE_e;
if (dComIfGp_demo_mode() == 1)
return dEvtMove_MOVE_e;
if (fopAcM_checkStatus(i_ac, fopAcStts_UNK800_e))
if (fopAcM_CheckStatus(i_ac, fopAcStts_UNK800_e))
return dEvtMove_MOVE_e;
if ((dComIfGp_event_getMode() == dEvtMode_COMPULSORY_e || dComIfGp_event_getMode() == dEvtMode_TALK_e) && fopAcM_checkStatus(i_ac, fopAcStts_BOSS_e))
if ((dComIfGp_event_getMode() == dEvtMode_COMPULSORY_e || dComIfGp_event_getMode() == dEvtMode_TALK_e) && fopAcM_CheckStatus(i_ac, fopAcStts_BOSS_e))
return dEvtMove_NOMOVE_e;
if (chkEventFlag(0x80) && fopAcM_checkStatus(i_ac, fopAcStts_BOSS_e))
if (chkEventFlag(0x80) && fopAcM_CheckStatus(i_ac, fopAcStts_BOSS_e))
return dEvtMove_NOMOVE_e;
if (fopAcM_checkStatus(i_ac, fopAcStts_UNK4000_e))
if (fopAcM_CheckStatus(i_ac, fopAcStts_UNK4000_e))
return dEvtMove_MOVE_e;
return fopAcM_checkStatus(i_ac, fopAcStts_CARRY_e) ? dEvtMove_MOVE_e : dEvtMove_NOMOVE_e;
return fopAcM_CheckStatus(i_ac, fopAcStts_CARRY_e) ? dEvtMove_MOVE_e : dEvtMove_NOMOVE_e;
}
/* 80071418-80071468 .text compulsory__14dEvt_control_cFPvPCcUs */
+1 -1
View File
@@ -397,7 +397,7 @@ int dEvent_manager_c::getMyStaffId(const char* name, fopAc_ac_c* actor, int tagI
if (mList.getHeaderP() == NULL)
return -1;
if (actor && !fopAcM_checkStatus(actor, fopAcStts_FORCEMOVE_e))
if (actor && !fopAcM_CheckStatus(actor, fopAcStts_FORCEMOVE_e))
return -1;
for (s32 i = 0; i < mList.getEventNum(); i++) {
+1 -1
View File
@@ -1704,7 +1704,7 @@ void dSnap_Obj::SetGeoSph(const Vec& center, f32 radius) {
/* 800CDB68-800CDB94 .text SetInf__9dSnap_ObjFUcPC10fopAc_ac_cUcUcs */
void dSnap_Obj::SetInf(u8 r4, const fopAc_ac_c* r5, u8 r6, u8 r7, s16 cullAngle) {
mPhoto = r4;
mActorPID = fpcM_GetID(r5);
mActorPID = fopAcM_GetID(const_cast<fopAc_ac_c*>(r5));
field_0x1a = r6;
field_0x19 = r7;
mCullAngle = cullAngle;
+7 -7
View File
@@ -41,13 +41,13 @@ s32 fopAc_Draw(void* pProc) {
(
moveApproval == dEvtMove_FORCE_e
|| (
!fopAcM_checkStatus(actor, fopAc_ac_c::getStopStatus())
!fopAcM_CheckStatus(actor, fopAc_ac_c::getStopStatus())
&& !(
fopAcM_checkStatus(actor, fopAcStts_CULL_e)
fopAcM_CheckStatus(actor, fopAcStts_CULL_e)
&& fopAcM_cullingCheck(actor)
)
)
) && !fopAcM_checkStatus(actor, fopAcStts_NODRAW_e)
) && !fopAcM_CheckStatus(actor, fopAcStts_NODRAW_e)
) {
fopAcM_OffCondition(actor, fopAcCnd_NODRAW_e);
ret = fpcLf_DrawMethod((leafdraw_method_class*)actor->sub_method, actor);
@@ -57,7 +57,7 @@ s32 fopAc_Draw(void* pProc) {
fopAcM_OffStatus(actor, fopAcStts_NODRAW_e);
if (dComIfGp_roomControl_getStayNo() >= 0 && fopAcM_checkStatus(actor, fopAcStts_SHOWMAP_e))
if (dComIfGp_roomControl_getStayNo() >= 0 && fopAcM_CheckStatus(actor, fopAcStts_SHOWMAP_e))
dMap_drawActorPointMiniMap(actor);
}
@@ -77,7 +77,7 @@ s32 fopAc_Execute(void* pProc) {
CHECK_FLOAT_CLASS(0x27f, actor->current.pos.z);
CHECK_VEC3_RANGE(0x286, actor->current.pos);
if (fopAcM_checkStatus(actor, fopAcStts_NOPAUSE_e) || (!dMenu_flag() && !dScnPly_ply_c::isPause())) {
if (fopAcM_CheckStatus(actor, fopAcStts_NOPAUSE_e) || (!dMenu_flag() && !dScnPly_ply_c::isPause())) {
actor->eventInfo.beforeProc();
s32 moveApproval = dComIfGp_event_moveApproval(actor);
@@ -88,10 +88,10 @@ s32 fopAc_Execute(void* pProc) {
|| (
(
moveApproval != dEvtMove_NOMOVE_e
&& !fopAcM_checkStatus(actor, fopAc_ac_c::getStopStatus())
&& !fopAcM_CheckStatus(actor, fopAc_ac_c::getStopStatus())
)
&& (
!fopAcM_checkStatus(actor, fopAcStts_NOCULLEXEC_e)
!fopAcM_CheckStatus(actor, fopAcStts_NOCULLEXEC_e)
|| !fopAcM_CheckCondition(actor, fopAcCnd_NODRAW_e)
)
)
+1 -1
View File
@@ -1326,7 +1326,7 @@ void fopAcM_setCarryNow(fopAc_ac_c* i_this, BOOL stageLayer) {
/* 80027E5C-80027ED8 .text fopAcM_cancelCarryNow__FP10fopAc_ac_c */
void fopAcM_cancelCarryNow(fopAc_ac_c* i_this) {
if (fopAcM_checkStatus(i_this, fopAcStts_CARRY_e)) {
if (fopAcM_CheckStatus(i_this, fopAcStts_CARRY_e)) {
fopAcM_OffStatus(i_this, fopAcStts_CARRY_e);
fopAcM_setRoomLayer(i_this, fopAcM_GetRoomNo(i_this));
i_this->shape_angle.z = 0;