From 1d6c9ed27912a4cdda28c65b4738045e6dd7afdf Mon Sep 17 00:00:00 2001
From: pajlada <rasmus.karlsson@pajlada.com>
Date: Sun, 5 Nov 2023 11:51:57 +0100
Subject: [PATCH] feat: forward logging from miniaudio (#4836)

These logs are posted to the `chatterino.sound` category
miniaudio debug -> debug
miniaudio info -> debug
miniaudio warning -> warning
miniaudio error -> warning
---
 src/controllers/sound/SoundController.cpp | 66 ++++++++++++++++++++++-
 1 file changed, 65 insertions(+), 1 deletion(-)

diff --git a/src/controllers/sound/SoundController.cpp b/src/controllers/sound/SoundController.cpp
index da7871a7950..5a527d0e197 100644
--- a/src/controllers/sound/SoundController.cpp
+++ b/src/controllers/sound/SoundController.cpp
@@ -12,6 +12,48 @@
 #include <limits>
 #include <memory>
 
+namespace {
+
+using namespace chatterino;
+
+void miniaudioLogCallback(void *userData, ma_uint32 level, const char *pMessage)
+{
+    (void)userData;
+
+    QString message{pMessage};
+
+    switch (level)
+    {
+        case MA_LOG_LEVEL_DEBUG: {
+            qCDebug(chatterinoSound).noquote()
+                << "ma debug:  " << message.trimmed();
+        }
+        break;
+        case MA_LOG_LEVEL_INFO: {
+            qCDebug(chatterinoSound).noquote()
+                << "ma info:   " << message.trimmed();
+        }
+        break;
+        case MA_LOG_LEVEL_WARNING: {
+            qCWarning(chatterinoSound).noquote()
+                << "ma warning:" << message.trimmed();
+        }
+        break;
+        case MA_LOG_LEVEL_ERROR: {
+            qCWarning(chatterinoSound).noquote()
+                << "ma error:  " << message.trimmed();
+        }
+        break;
+        default: {
+            qCWarning(chatterinoSound).noquote()
+                << "ma unknown:" << message.trimmed();
+        }
+        break;
+    }
+}
+
+}  // namespace
+
 namespace chatterino {
 
 // NUM_SOUNDS specifies how many simultaneous default ping sounds & decoders to create
@@ -31,8 +73,30 @@ void SoundController::initialize(Settings &settings, Paths &paths)
 
     ma_result result{};
 
+    // We are leaking this log object on purpose
+    auto *logger = new ma_log;
+
+    result = ma_log_init(nullptr, logger);
+    if (result != MA_SUCCESS)
+    {
+        qCWarning(chatterinoSound) << "Error initializing logger:" << result;
+        return;
+    }
+
+    result = ma_log_register_callback(
+        logger, ma_log_callback_init(miniaudioLogCallback, nullptr));
+    if (result != MA_SUCCESS)
+    {
+        qCWarning(chatterinoSound)
+            << "Error registering logger callback:" << result;
+        return;
+    }
+
+    auto contextConfig = ma_context_config_init();
+    contextConfig.pLog = logger;
+
     /// Initialize context
-    result = ma_context_init(nullptr, 0, nullptr, this->context.get());
+    result = ma_context_init(nullptr, 0, &contextConfig, this->context.get());
     if (result != MA_SUCCESS)
     {
         qCWarning(chatterinoSound) << "Error initializing context:" << result;