From f8e615fae73c2cdba2f68a18de8b365ca18a0b81 Mon Sep 17 00:00:00 2001
From: LagoLunatic <LagoLunatic@users.noreply.github.com>
Date: Tue, 22 Oct 2024 12:36:12 -0400
Subject: [PATCH] cleanup

---
 include/JSystem/JAudio/JASTrack.h   |  4 ++++
 src/JSystem/JAudio/JASSeqParser.cpp | 15 ++++++---------
 src/d/actor/d_a_obj_ebomzo.cpp      |  2 +-
 src/d/actor/d_a_sail.cpp            |  2 +-
 src/d/d_chain.cpp                   |  2 +-
 src/m_Do/m_Do_audio.cpp             |  2 ++
 6 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/include/JSystem/JAudio/JASTrack.h b/include/JSystem/JAudio/JASTrack.h
index 9a518f16f..fdb0a50c8 100644
--- a/include/JSystem/JAudio/JASTrack.h
+++ b/include/JSystem/JAudio/JASTrack.h
@@ -135,10 +135,14 @@ namespace JASystem {
         public:
             TimedParam_();
 
+#ifdef __MWERKS__
             union {
                 AInnerParam_ mInnerParam;
                 MoveParam_ mMoveParams[TIMED_Count];
             };
+#else
+            MoveParam_ mMoveParams[TIMED_Count];
+#endif
         };
 
         class TNoteMgr {
diff --git a/src/JSystem/JAudio/JASSeqParser.cpp b/src/JSystem/JAudio/JASSeqParser.cpp
index 54b49cde9..28ed200bd 100644
--- a/src/JSystem/JAudio/JASSeqParser.cpp
+++ b/src/JSystem/JAudio/JASSeqParser.cpp
@@ -577,10 +577,8 @@ int JASystem::TSeqParser::cmdVibPitch(TTrack* track, u32* args) {
 /* 8027F544-8027F5C8       .text cmdIIRSet__Q28JASystem10TSeqParserFPQ28JASystem6TTrackPUl */
 int JASystem::TSeqParser::cmdIIRSet(TTrack* track, u32* args) {
     for (u8 i = 0; i < 4; i++) {
-        // TTrack::MoveParam_* iir = &track->mTimedParam.mInnerParam.mIIRs[i];
-        // Fakematch? Accessing mIIRs directly results in fewer instructions than indexing into
-        // mVolume as if it was an array.
-        TTrack::MoveParam_* iir = (&track->mTimedParam.mInnerParam.mVolume) + (u8)(i + 0xC);
+        u8 iirIndex = TTrack::TIMED_IIR_Unk0 + i;
+        TTrack::MoveParam_* iir = &track->mTimedParam.mMoveParams[iirIndex];
         iir->mTargetValue = (s16)args[i] / 32768.0f;
         iir->mCurrentValue = iir->mTargetValue;
         iir->mMoveAmount = 0.0f;
@@ -594,10 +592,8 @@ int JASystem::TSeqParser::cmdIIRCutOff(TTrack* track, u32* args) {
     u8 iirTableIdx = args[0];
     for (u8 i = 0; i < 4; i++) {
         s16* table = &JASystem::Player::CUTOFF_TO_IIR_TABLE[iirTableIdx * 4];
-        // TTrack::MoveParam_* iir = &track->mTimedParam.mInnerParam.mIIRs[i];
-        // Fakematch? Accessing mIIRs directly results in fewer instructions than indexing into
-        // mVolume as if it was an array.
-        TTrack::MoveParam_* iir = (&track->mTimedParam.mInnerParam.mVolume) + (u8)(i + 0xC);
+        u8 iirIndex = TTrack::TIMED_IIR_Unk0 + i;
+        TTrack::MoveParam_* iir = &track->mTimedParam.mMoveParams[iirIndex];
         iir->mTargetValue = table[i] / (32768.0f - 1.0f);
         iir->mCurrentValue = iir->mTargetValue;
         iir->mMoveAmount = 0.0f;
@@ -761,7 +757,7 @@ int JASystem::TSeqParser::cmdSetParam(TTrack* track, u8 param_2) {
     case 4:
         data = track->getSeq()->readByte();
         break;
-    case 8:
+    case 8: {
         u8 byte = track->getSeq()->readByte();
         if (byte & 0x80) {
             data = byte << 8;
@@ -769,6 +765,7 @@ int JASystem::TSeqParser::cmdSetParam(TTrack* track, u8 param_2) {
             data = byte << 8 | byte << 1;
         }
         break;
+    }
     case 12:
         data = track->getSeq()->read16();
         break;
diff --git a/src/d/actor/d_a_obj_ebomzo.cpp b/src/d/actor/d_a_obj_ebomzo.cpp
index 92a97a258..6159ba52f 100644
--- a/src/d/actor/d_a_obj_ebomzo.cpp
+++ b/src/d/actor/d_a_obj_ebomzo.cpp
@@ -76,7 +76,7 @@ int daObjEbomzo::Act_c::Create() {
     }
     else mMode = Ebomzo_Mode_Check;
 
-  return TRUE;
+    return TRUE;
 }
 
 /* 0000021C-000003A0       .text Mthd_Create__Q211daObjEbomzo5Act_cFv */
diff --git a/src/d/actor/d_a_sail.cpp b/src/d/actor/d_a_sail.cpp
index 44dd18de9..2d403d843 100644
--- a/src/d/actor/d_a_sail.cpp
+++ b/src/d/actor/d_a_sail.cpp
@@ -107,7 +107,7 @@ Vec l_pos[] = {
     {840.0f, -1100.0f, 0.0f},
 };
 
-f32 l_texCoord[][2] = {
+cXy l_texCoord[] = {
     {0.0f, 0.0f},
     {0.165f, 0.0f},
     {0.33f, 0.0f},
diff --git a/src/d/d_chain.cpp b/src/d/d_chain.cpp
index a5b3cd49b..3314f26af 100644
--- a/src/d/d_chain.cpp
+++ b/src/d/d_chain.cpp
@@ -75,7 +75,7 @@ static Vec l_pos[] = {
     { 2.0f, -1.522254f, 7.0f },
 
 };
-static f32 l_texCoord[][2] = {
+static cXy l_texCoord[] = {
     { 0.02736f, 0.041406f },
     { 0.97264f, 0.041406f },
     { 0.830848f, 0.40324f },
diff --git a/src/m_Do/m_Do_audio.cpp b/src/m_Do/m_Do_audio.cpp
index 76b414229..9373cdd6a 100644
--- a/src/m_Do/m_Do_audio.cpp
+++ b/src/m_Do/m_Do_audio.cpp
@@ -8,7 +8,9 @@
 #include "JSystem/JAudio/JAISequenceMgr.h"
 #include "JSystem/JAudio/JAIStreamMgr.h"
 #include "JSystem/JKernel/JKRSolidHeap.h"
+#if VERSION == VERSION_JPN
 #include "JSystem/JUtility/JUTConsole.h"
+#endif
 #include "SSystem/SComponent/c_lib.h"
 #include "d/d_com_inf_game.h"
 #include "m_Do/m_Do_controller_pad.h"