diff --git a/src/overlays/actors/ovl_En_S_Goro/z_en_s_goro.c b/src/overlays/actors/ovl_En_S_Goro/z_en_s_goro.c index 29704afa02..c91c100145 100644 --- a/src/overlays/actors/ovl_En_S_Goro/z_en_s_goro.c +++ b/src/overlays/actors/ovl_En_S_Goro/z_en_s_goro.c @@ -884,18 +884,19 @@ void EnSGoro_UpdateToIdleAnimation(EnSGoro* this) { } void EnSGoro_UpdateCollider(EnSGoro* this, PlayState* play) { - Vec3f world_pos = this->actor.world.pos; + Vec3f worldPos = this->actor.world.pos; f32 radius = 24.0f; f32 height = 62.0f; - this->collider.dim.pos.x = world_pos.x; - this->collider.dim.pos.y = world_pos.y; - this->collider.dim.pos.z = world_pos.z; + this->collider.dim.pos.x = worldPos.x; + this->collider.dim.pos.y = worldPos.y; + this->collider.dim.pos.z = worldPos.z; this->collider.dim.radius = radius; this->collider.dim.height = height; - //! @bug: It is not clear what this is for. - if ((s32)this != -0x190) { + //! @bug: The check is useless. If &this->collider somehow was NULL the above code would have already dereferenced + //! it. + if (&this->collider != NULL) { CollisionCheck_SetOC(play, &play->colChkCtx, &this->collider.base); } }