From 0ee416c53896590d8d1a4b0c800e849cf133f868 Mon Sep 17 00:00:00 2001 From: LagoLunatic Date: Sun, 25 May 2025 17:28:35 -0400 Subject: [PATCH] Match daPy_lk_c::checkPlayerGuard, switch all of daPy_lk_c's inlines to return BOOL instead of bool --- include/d/actor/d_a_player_main.h | 38 +++++++++++++++---------------- src/d/actor/d_a_player_main.cpp | 7 +----- 2 files changed, 20 insertions(+), 25 deletions(-) diff --git a/include/d/actor/d_a_player_main.h b/include/d/actor/d_a_player_main.h index 59a10809c..249024e3e 100644 --- a/include/d/actor/d_a_player_main.h +++ b/include/d/actor/d_a_player_main.h @@ -1731,32 +1731,32 @@ public: J3DAnmTextureSRTKey* getIceArrowBtk() { return mpIceArrowBtk; } J3DAnmTextureSRTKey* getLightArrowBtk() { return mpLightArrowBtk; } - bool checkUpperAnime(u16 i_idx) const { return m_anm_heap_upper[UPPER_MOVE2_e].mIdx == i_idx; } - bool checkNoUpperAnime() const { return m_anm_heap_upper[UPPER_MOVE2_e].mIdx == 0xFFFF; } + BOOL checkUpperAnime(u16 i_idx) const { return m_anm_heap_upper[UPPER_MOVE2_e].mIdx == i_idx; } + BOOL checkNoUpperAnime() const { return m_anm_heap_upper[UPPER_MOVE2_e].mIdx == 0xFFFF; } - bool checkGrabAnime() const { return checkGrabAnimeLight() || checkGrabAnimeHeavy(); }; - bool checkGrabAnimeLight() const { return checkUpperAnime(LKANM_BCK_GRABWAIT); }; - bool checkGrabAnimeHeavy() const { return checkUpperAnime(LKANM_BCK_GRABWAITB); }; - bool checkBoomerangCatchAnime() const { return checkUpperAnime(LKANM_BCK_BOOMCATCH); }; - bool checkBoomerangThrowAnime() const { return checkUpperAnime(LKANM_BCK_BOOMTHROW); }; - bool checkBoomerangReadyAnime() const { return checkUpperAnime(LKANM_BCK_BOOMWAIT); }; - bool checkHookshotReadyAnime() const { return checkUpperAnime(LKANM_BCK_HOOKSHOTWAIT); } - bool checkDashDamageAnime() const { return checkUpperAnime(LKANM_BCK_DAMDASH); } - bool checkBowReloadAnime() const { return checkUpperAnime(LKANM_BCK_ARROWRELORD); } - bool checkBowShootAnime() const { return checkUpperAnime(LKANM_BCK_ARROWSHOOT); } - bool checkBowWaitAnime() const { return checkUpperAnime(LKANM_BCK_BOWWAIT); } - bool checkGuardSlip() const { + BOOL checkGrabAnime() const { return checkGrabAnimeLight() || checkGrabAnimeHeavy(); }; + BOOL checkGrabAnimeLight() const { return checkUpperAnime(LKANM_BCK_GRABWAIT); }; + BOOL checkGrabAnimeHeavy() const { return checkUpperAnime(LKANM_BCK_GRABWAITB); }; + BOOL checkBoomerangCatchAnime() const { return checkUpperAnime(LKANM_BCK_BOOMCATCH); }; + BOOL checkBoomerangThrowAnime() const { return checkUpperAnime(LKANM_BCK_BOOMTHROW); }; + BOOL checkBoomerangReadyAnime() const { return checkUpperAnime(LKANM_BCK_BOOMWAIT); }; + BOOL checkHookshotReadyAnime() const { return checkUpperAnime(LKANM_BCK_HOOKSHOTWAIT); } + BOOL checkDashDamageAnime() const { return checkUpperAnime(LKANM_BCK_DAMDASH); } + BOOL checkBowReloadAnime() const { return checkUpperAnime(LKANM_BCK_ARROWRELORD); } + BOOL checkBowShootAnime() const { return checkUpperAnime(LKANM_BCK_ARROWSHOOT); } + BOOL checkBowWaitAnime() const { return checkUpperAnime(LKANM_BCK_BOWWAIT); } + BOOL checkGuardSlip() const { return mCurProc == daPyProc_GUARD_SLIP_e || mCurProc == daPyProc_CROUCH_DEFENSE_SLIP_e; } - bool checkUpperGuardAnime() const { + BOOL checkUpperGuardAnime() const { return checkUpperAnime(LKANM_BCK_ATNG) || checkUpperAnime(LKANM_BCK_ATNGHAM); } s16 checkTinkleShield() const { return mTinkleShieldTimer; } void setTinkleShield(s16 time) { mTinkleShieldTimer = time; } - bool checkNoDamageMode() const { return checkEquipDragonShield() || checkTinkleShield() != 0; } + BOOL checkNoDamageMode() const { return checkEquipDragonShield() || checkTinkleShield() != 0; } s16 checkTinkleHover() const { return mTinkleHoverTimer; } void setHoverBoots(s16 time) { onNoResetFlg0(daPyFlg0_HOVER_BOOTS); @@ -1764,12 +1764,12 @@ public: } void onShipTact() { onNoResetFlg1(daPyFlg1_SHIP_TACT); } void offShipTact() { offNoResetFlg1(daPyFlg1_SHIP_TACT); } - bool checkShipGetOff() { return mCurProc == daPyProc_SHIP_GET_OFF_e; } + BOOL checkShipGetOff() { return mCurProc == daPyProc_SHIP_GET_OFF_e; } void onShipDrop(s16 param_1) { onNoResetFlg0(daPyFlg0_SHIP_DROP); m3550 = param_1; } - bool checkCarryActionNow() const { + BOOL checkCarryActionNow() const { return mCurProc == daPyProc_GRAB_PUT_e || mCurProc == daPyProc_GRAB_UP_e || mCurProc == daPyProc_GRAB_THROW_e; @@ -1792,7 +1792,7 @@ public: mActivePlayerBombs--; } } - bool checkSwordEquip() const { + BOOL checkSwordEquip() const { return dComIfGs_getSelectEquip(0) != dItem_NONE_e || checkSwordMiniGame(); } diff --git a/src/d/actor/d_a_player_main.cpp b/src/d/actor/d_a_player_main.cpp index b23aeaa01..ab6e78792 100644 --- a/src/d/actor/d_a_player_main.cpp +++ b/src/d/actor/d_a_player_main.cpp @@ -1024,12 +1024,7 @@ void daPy_lk_c::playTextureAnime() { /* 8010552C-8010558C .text checkPlayerGuard__9daPy_lk_cCFv */ BOOL daPy_lk_c::checkPlayerGuard() const { - /* Nonmatching */ - bool guard = false; - if (mCurProc == daPyProc_CROUCH_DEFENSE_e || checkUpperGuardAnime() || checkGuardSlip()) { - guard = true; - } - return guard; + return mCurProc == daPyProc_CROUCH_DEFENSE_e || checkUpperGuardAnime() || checkGuardSlip(); } /* 8010558C-801056E4 .text setOutPower__9daPy_lk_cFfsi */