better error handling

Author: maxnut

CMakeLists.txt

@@ -14,6 +14,7 @@ project(ffmpeg-api VERSION 1.0.0)
 add_library(${PROJECT_NAME} SHARED
     src/recorder.cpp
     src/audio_mixer.cpp
+    src/utils.cpp
     # Add any extra C++ source files here
 )
 

include/audio_mixer.hpp

@@ -2,6 +2,8 @@
 
 #include "export.hpp"
 
+#include <Geode/Result.hpp>
+
 #include <filesystem>
 
 namespace ffmpeg {
@@ -21,7 +23,7 @@ class FFMPEG_API_DLL AudioMixer {
      * @warning The audio file is expected to contain stereo (dual-channel) audio. Using other formats might lead to unexpected results.
      * @warning The video file is expected to contain a single video stream. Only the first video stream will be copied.
      */
-    void mixVideoAudio(std::filesystem::path videoFile, std::filesystem::path audioFile, std::filesystem::path outputMp4File);
+    geode::Result<void> mixVideoAudio(std::filesystem::path videoFile, std::filesystem::path audioFile, std::filesystem::path outputMp4File);
 
     /**
      * @deprecated sampleRate parameter is no longer used. Use the other overload of this function instead.
@@ -54,7 +56,7 @@ class FFMPEG_API_DLL AudioMixer {
      * @warning The raw audio data is expected to be stereo (dual-channel). Using mono or multi-channel audio might lead to issues.
      * @warning The video file is expected to contain a single video stream. Only the first video stream will be copied.
      */
-    void mixVideoRaw(const std::filesystem::path& videoFile, const std::vector<float>& raw, const std::filesystem::path &outputMp4File);
+    geode::Result<void> mixVideoRaw(const std::filesystem::path& videoFile, const std::vector<float>& raw, const std::filesystem::path &outputMp4File);
 };
 
 }

include/recorder.hpp

@@ -3,6 +3,8 @@
 #include "render_settings.hpp"
 #include "export.hpp"
 
+#include <Geode/Result.hpp>
+
 #include <vector>
 #include <string>
 #include <memory>
@@ -35,7 +37,7 @@ class FFMPEG_API_DLL Recorder {
      * 
      * @return true if initialization is successful, false otherwise.
      */
-    bool init(const RenderSettings& settings);
+    geode::Result<void> init(const RenderSettings& settings);
     /**
      * @brief Stops the recording process and finalizes the output file.
      *
@@ -57,7 +59,7 @@ class FFMPEG_API_DLL Recorder {
      * 
      * @warning Ensure that the frameData size matches the expected dimensions of the frame.
      */
-    bool writeFrame(const std::vector<uint8_t>& frameData);
+    geode::Result<void> writeFrame(const std::vector<uint8_t>& frameData);
 
     /**
      * @brief Retrieves a list of available codecs for video encoding.
@@ -70,7 +72,7 @@ class FFMPEG_API_DLL Recorder {
     std::vector<std::string> getAvailableCodecs();
 
 private:
-    void filterFrame(AVFrame* inputFrame, AVFrame* outputFrame);
+    geode::Result<void> filterFrame(AVFrame* inputFrame, AVFrame* outputFrame);
 
 private:
     AVFormatContext* m_formatContext = nullptr;

mod.json

@@ -13,7 +13,7 @@
     },
     "id": "eclipse.ffmpeg-api",
     "name": "FFmpeg API",
-    "version": "v1.1.2",
+    "version": "v1.1.3",
     "developers": ["Eclipse Team", "maxnu"],
     "description": "Interaction with FFmpeg made easy",
     "tags": ["utility", "offline", "developer"],