Audio Engine Fixes

Src/IAEngine/imp/cpp/Audio.cpp

@@ -19,12 +19,26 @@
 #include <SDL3/SDL_iostream.h>
 #include <SDL3_mixer/SDL_mixer.h>
 
+#include <condition_variable>
 #include <thread>
 
 namespace ia::iae
 {
+    struct TrackData
+    {
+        BOOL IsActive{};
+        MIX_Track *Track{};
+        INT32 LoopTimes{};
+        TimePeriod LoopDelay{};
+        std::thread *PlaybackThread;
+
+        std::mutex Mutex;
+        std::condition_variable CV;
+    };
+
     MIX_Mixer *g_mixer{};
-    Vector<MIX_Track *> g_tracks;
+    Vector<TrackData*> g_tracks;
+    Vector<INT64> g_destroyQueue;
     Vector<MIX_Audio *> g_audioData;
 
     VOID Audio::Initialize()
@@ -38,7 +52,48 @@ namespace ia::iae
 
     VOID Audio::Terminate()
     {
+        for (SIZE_T i = 0; i < g_tracks.size(); i++)
+        {
+            {
+                std::lock_guard<std::mutex> lock(g_tracks[i]->Mutex);
+                g_tracks[i]->IsActive = false;
+            }
+            g_tracks[i]->CV.notify_one();
+            g_tracks[i]->PlaybackThread->join();
+            delete g_tracks[i];
+        }
+        MIX_StopAllTracks(g_mixer, 0);
         g_audioData.clear();
+        g_tracks.clear();
+    }
+
+    VOID Audio::Update()
+    {
+        for (SIZE_T i = 0; i < g_tracks.size(); i++)
+        {
+            auto &t = g_tracks[i];
+            if (!t->IsActive && !MIX_TrackPlaying(t->Track))
+                DestroyTrack(i);
+        }
+
+        // [IATODO]: IMPL: Destroy queue
+    }
+
+    INT64 Audio::CreateTrack()
+    {
+        g_tracks.pushBack(new TrackData{true, MIX_CreateTrack(g_mixer)});
+        return g_tracks.size() - 1;
+    }
+
+    VOID Audio::DestroyTrack(IN INT64 trackHandle)
+    {
+        if(trackHandle >= g_tracks.size())
+            return;
+        auto& t = g_tracks[trackHandle];
+        ClearTrack(trackHandle);
+        MIX_StopTrack(t->Track, 0);
+        t->IsActive = false;
+        g_destroyQueue.pushBack(trackHandle);
     }
 
     Sound Audio::CreateSound(IN PCUINT8 audioData, IN SIZE_T audioDataSize)
@@ -47,62 +102,55 @@ namespace ia::iae
         return Sound(g_audioData.size() - 1);
     }
 
-    INT64 Audio::CreateTrack()
-    {
-        g_tracks.pushBack(MIX_CreateTrack(g_mixer));
-        return g_tracks.size() - 1;
-    }
-
-    VOID Audio::DestroyTrack(IN INT64 trackHandle)
-    {
-        ClearTrack(trackHandle);
-    }
-
     VOID Audio::QueueTrackData(IN INT64 trackHandle, IN INT64 dataHandle)
     {
-        MIX_SetTrackAudio(g_tracks[trackHandle], g_audioData[dataHandle]);
+        MIX_SetTrackAudio(g_tracks[trackHandle]->Track, g_audioData[dataHandle]);
     }
 
     VOID Audio::PlayTrack(IN INT64 trackHandle, IN INT32 loopTimes, IN TimePeriod loopDelay)
     {
-        struct __callback_data
+        auto &t = g_tracks[trackHandle];
-        {
+        t->LoopTimes = loopTimes;
-            INT32 loopTimes;
+        t->LoopDelay = loopDelay;
-            TimePeriod loopDelay;
+        if (t->LoopTimes == 0)
-        };
+            t->IsActive = false;
-
-        const auto callbackData = new __callback_data{.loopTimes = loopTimes, .loopDelay = loopDelay};
-
-        MIX_PlayTrack(g_tracks[trackHandle], 0);
-        MIX_SetTrackStoppedCallback(
-            g_tracks[trackHandle],
-            [](PVOID _callbackData, MIX_Track *track) {
-                const auto callbackData = (__callback_data *) _callbackData;
-                if (callbackData->loopTimes < 0)
-                    goto loop_next_iteration;
-                else if (callbackData->loopTimes == 0)
-                    return;
         else
-                    callbackData->loopTimes -= 1;
+            t->PlaybackThread = new std::thread([trackHandle]() {
+                auto &t = g_tracks[trackHandle];
+                while (t->IsActive)
+                {
+                    if (MIX_TrackPlaying(t->Track))
+                        continue;
+                    if (t->LoopTimes < 0)
+                        goto loop_next_iteration;
+                    else if (!t->LoopTimes)
+                    {
+                        t->IsActive = false;
+                        return;
+                    }
+                    t->LoopTimes--;
                 loop_next_iteration:
-                std::this_thread::sleep_for(std::chrono::milliseconds((INT32) (callbackData->loopDelay.GetValue() * 1000)));
+                    std::unique_lock<std::mutex> lock(t->Mutex);
-                MIX_PlayTrack(track, 0);
+                    t->CV.wait_for(lock, std::chrono::milliseconds((INT32) (t->LoopDelay.GetValue() * 1000)),
-            },
+                                [t]() { return !t->IsActive; });
-            callbackData);
+                    MIX_PlayTrack(t->Track, 0);
+                }
+            });
+        MIX_PlayTrack(t->Track, 0);
     }
 
     VOID Audio::ClearTrack(IN INT64 trackHandle)
     {
-        MIX_StopTrack(g_tracks[trackHandle], 0);
+        MIX_StopTrack(g_tracks[trackHandle]->Track, 0);
     }
 
     VOID Audio::PauseTrack(IN INT64 trackHandle)
     {
-        MIX_PauseTrack(g_tracks[trackHandle]);
+        MIX_PauseTrack(g_tracks[trackHandle]->Track);
     }
 
     VOID Audio::ResumeTrack(IN INT64 trackHandle)
     {
-        MIX_ResumeTrack(g_tracks[trackHandle]);
+        MIX_ResumeTrack(g_tracks[trackHandle]->Track);
     }
 } // namespace ia::iae

Src/IAEngine/imp/cpp/Components/Physics.cpp

@@ -35,6 +35,7 @@ namespace ia::iae
 
     VOID PhysicsComponent::Update()
     {
+        m_velocity = {};
     }
 
     Handle PhysicsComponent::CreateCollider()
@@ -58,6 +59,7 @@ namespace ia::iae
     {
         IA_ASSERT(m_isDynamic);
         const auto v = direction * m_movementSpeed;
+        m_velocity += v;
         for(const auto& t: m_colliders)
         {
             if(!Physics::CanMove(m_physicsHandle, t, v))

Src/IAEngine/imp/cpp/IAEngine.cpp

@@ -138,6 +138,7 @@ namespace ia::iae
             g_physicsDebugDrawEnabled = !g_physicsDebugDrawEnabled;
 
         Physics::Update();
+        Audio::Update();
 
         if B_LIKELY (g_activeScene)
             g_activeScene->Update();

Src/IAEngine/imp/cpp/Nodes/Node.cpp

@@ -72,9 +72,10 @@ namespace ia::iae
             c->Disable();
     }
 
-    VOID Node::AddChild(IN RefPtr<Node> node)
+    Node* Node::AddChild(IN RefPtr<Node> node)
     {
         m_children.pushBack(node);
+        return node.get();
     }
 
     VOID Node::AddComponent(IN RefPtr<IComponent> component)

Src/IAEngine/imp/cpp/Physics/Physics.cpp

@@ -17,6 +17,8 @@
 #include <IAEngine/IAEngine.hpp>
 #include <IAEngine/Physics/Physics.hpp>
 #include <IAEngine/Rendering/DebugDraw.hpp>
+#include <IAEngine/Rendering/Camera.hpp>
+#include <IAEngine/Rendering/Renderer.hpp>
 
 #include <IAEngine/Nodes/Node.hpp>
 
@@ -28,49 +30,6 @@ namespace ia::iae
 
     Vector<PhysicsComponent *> g_physicsComponents;
 
-    FLOAT32 Sq(IN FLOAT32 v)
-    {
-        return v * v;
-    }
-
-    enum class RectSide
-    {
-        NONE,
-        LEFT,
-        RIGHT,
-        TOP,
-        BOTTOM,
-    };
-
-    RectSide GetRectSide(IN glm::vec2 d)
-    {
-        const auto dX = abs(d.x);
-        const auto dY = abs(d.y);
-        if (d.x >= 0)
-        {
-            if (dX > dY)
-            {
-                return RectSide::RIGHT;
-            }
-            else
-            {
-                return (d.y < 0) ? RectSide::TOP : RectSide::BOTTOM;
-            }
-        }
-        else
-        {
-            if (dX > dY)
-            {
-                return RectSide::LEFT;
-            }
-            else
-            {
-                return (d.y < 0) ? RectSide::TOP : RectSide::BOTTOM;
-            }
-        }
-        return RectSide::NONE;
-    }
-
     VOID Physics::Initialize()
     {
     }
@@ -95,7 +54,7 @@ namespace ia::iae
                 auto color = glm::vec4{0.75f, 0.0f, 0.0f, 1.0f};
                 if(c.IsTrigger)
                     color = {0.25f, 0.45f, 0.75f, 0.75f};
-                DebugDraw::DrawRect(t->GetNode()->GetPosition() + c.Position, c.Size, color, 2.0f);
+                DebugDraw::DrawRect(t->GetNode()->GetPosition() + c.Position - Renderer::GetCamera()->Position(), c.Size, color, 2.0f);
             }
         }
     }

Src/IAEngine/inc/IAEngine/Audio.hpp

@@ -64,11 +64,13 @@ namespace ia::iae
         STATIC VOID Initialize();
         STATIC VOID Terminate();
         
-        STATIC Sound CreateSound(IN PCUINT8 audioData, IN SIZE_T audioDataSize);
+        STATIC VOID Update();
 
         STATIC INT64 CreateTrack();
         STATIC VOID DestroyTrack(IN INT64 trackHandle);
 
+        STATIC Sound CreateSound(IN PCUINT8 audioData, IN SIZE_T audioDataSize);
+
         STATIC VOID QueueTrackData(IN INT64 trackHandle, IN INT64 dataHandle);
         STATIC VOID PlayTrack(IN INT64 trackHandle, IN INT32 loopTimes, IN TimePeriod loopDelay);
         STATIC VOID ClearTrack(IN INT64 trackHandle);

Src/IAEngine/inc/IAEngine/Components/Physics.hpp

@@ -58,14 +58,20 @@ namespace ia::iae
             return m_colliders;
         }
 
+        CONST glm::vec2& GetVelocity() CONST
+        {
+            return m_velocity;
+        }
+
       public:
         VOID Draw();
         VOID Update();
 
       private:
+        glm::vec2 m_velocity{};
         BOOL m_isDynamic{false};
-        FLOAT32 m_movementSpeed{2.0f};
         Vector<Collider> m_colliders;
+        FLOAT32 m_movementSpeed{1.0f};
         Handle m_physicsHandle{INVALID_HANDLE};
 
         VOID OnCollision(IN PhysicsComponent *other);

Src/IAEngine/inc/IAEngine/Input.hpp

@@ -255,8 +255,8 @@ namespace ia::iae
 
         enum class DirectionalInput: UINT8
         {
-            NONE = 0,
+            NONE = 255,
-            DOWN,
+            DOWN = 0,
             DOWN_LEFT,
             LEFT,
             UP_LEFT,

Src/IAEngine/inc/IAEngine/Nodes/Node.hpp

@@ -39,7 +39,7 @@ namespace ia::iae
         VIRTUAL VOID Disable();
 
       public:
-        VOID AddChild(IN RefPtr<Node> node);
+        Node* AddChild(IN RefPtr<Node> node);
 
         template<typename _component_type> _component_type *AddComponent();