Added basic error handling

CMakeLists.txt

@@ -20,8 +20,9 @@ pkg_check_modules(AVUTIL REQUIRED libavutil)
 pkg_check_modules(SWRESAMPLE REQUIRED libswresample)
 
 add_library(
-  ChargeAudio SHARED "src/ChargeAudio.hpp" "src/Engine.cpp" "src/Sound.cpp"
+  ChargeAudio SHARED
-                     "src/Listener.cpp" "lib/miniaudio/miniaudio.c")
+  "src/ChargeAudio.hpp" "src/Engine.cpp" "src/Sound.cpp" "src/Listener.cpp"
+  "src/ErrorHandling.cpp" "lib/miniaudio/miniaudio.c")
 
 target_link_libraries(
   ChargeAudio

src/ChargeAudio.hpp

@@ -114,5 +114,8 @@ private:
   friend class Sound;
 };
 
+void ThrowOnRuntimeError(std::string message,
+                         ma_result errorType = ma_result::MA_ERROR);
+
 } // namespace ChargeAudio
 #endif

src/Engine.cpp

@@ -1,9 +1,7 @@
 #include "ChargeAudio.hpp"
 #include "miniaudio/miniaudio.h"
 
-#include <Corrade/Utility/Debug.h>
 #include <cstdint>
-#include <stdexcept>
 
 using namespace ChargeAudio;
 using namespace Corrade;
@@ -13,11 +11,8 @@ Engine::Engine(uint32_t sampleRate, uint32_t channels) {
   maConfig.sampleRate = sampleRate;
   maConfig.channels = channels;
 
-  if ((maResponse = ma_engine_init(&maConfig, &maEngine)) != MA_SUCCESS) {
+  ThrowOnRuntimeError("Failed to init miniaudio engine",
-    Utility::Error{} << "Could not init miniaudio (" << maResponse << ")";
+                      ma_engine_init(&maConfig, &maEngine));
-    throw new std::runtime_error(
-        "Failed to init miniaudio engine. Check STDERR for more info.");
-  }
 }
 
 Engine::~Engine() { ma_engine_uninit(&maEngine); }
@@ -34,12 +29,7 @@ SoundContainer Engine::CreateSound(int bufferLengthInSeconds) {
         ma_result result = ma_pcm_rb_init(
             ma_format_s32, channels, sampleRate * channels * length, nullptr,
             nullptr, &sound->maRingBuffer);
-        if (result != MA_SUCCESS) {
+        ThrowOnRuntimeError("Failed to create a new ring buffer!", result);
-          Utility::Error{} << "Failed to create a new ring buffer!" << " ("
-                           << result << ")";
-          throw new std::runtime_error("Failed to create a new ring buffer! "
-                                       "Check STDERR for more info.");
-        }
       },
       Sound::SoundType::StreamedRawPCM,
       "Failed to create the sound from ring buffer: "));
@@ -56,13 +46,7 @@ SoundContainer Engine::CreateSound(uint8_t *data, int length) {
         ma_result result =
             ma_audio_buffer_init_copy(&config, &sound->maAudioBuffer);
 
-        if (result != MA_SUCCESS) {
+        ThrowOnRuntimeError("Failed to create a new audio buffer!", result);
-          Utility::Error{} << "Failed to create a new audio buffer!" << " ("
-                           << result << ")";
-          throw new std::runtime_error("Failed to create a new audio buffer! "
-                                       "Check STDERR for more info.");
-        }
-
         sound->maConfig.pDataSource = &sound->maAudioBuffer;
       },
       Sound::SoundType::RawPCM,

src/ErrorHandling.cpp

@@ -0,0 +1,12 @@
+#include "ChargeAudio.hpp"
+#include <stdexcept>
+
+void ChargeAudio::ThrowOnRuntimeError(std::string message,
+                                      ma_result errorType) {
+  if (errorType == MA_SUCCESS) {
+    return;
+  }
+
+  Utility::Error{} << message << " (" << errorType << ")";
+  throw new std::runtime_error(message + ". Check STDERR for more info.\n");
+}

src/Sound.cpp

@@ -1,7 +1,6 @@
 #include "ChargeAudio.hpp"
 
 #include <functional>
-#include <stdexcept>
 
 using namespace ChargeAudio;
 Sound::Sound(Engine *engine, std::function<void(Sound *)> setupFunction,
@@ -13,13 +12,8 @@ Sound::Sound(Engine *engine, std::function<void(Sound *)> setupFunction,
   setupFunction(this);
   ma_result maResponse =
       ma_sound_init_ex(&baseEngine->maEngine, &maConfig, &maSound);
-  if (maResponse != MA_SUCCESS) {
+  ThrowOnRuntimeError("Failed to create a new sound. " + additionalErrorMessage,
-    Utility::Error{} << "Failed to create a new sound" << " (" << maResponse
+                      maResponse);
-                     << ")";
-    throw new std::runtime_error(
-        "Failed to create a new sound. Check STDERR for more info.\n" +
-        additionalErrorMessage);
-  }
   type = soundType;
 }
 
@@ -78,7 +72,13 @@ void Sound::Pause() {
   state = Sound::SoundState::Paused;
 }
 
-void Sound::Reset() { ma_sound_seek_to_pcm_frame(&maSound, 0); }
+void Sound::Reset() {
+  if (type == SoundType::StreamedRawPCM) {
+    ThrowOnRuntimeError("You cannot reset on Streamed RawPCM sounds!");
+  }
+
+  ma_sound_seek_to_pcm_frame(&maSound, 0);
+}
 
 void Sound::SetPosition(Magnum::Vector3 position) {
   ma_sound_set_position(&maSound, position.x(), position.y(), position.z());