Audio module reviewed for standalone usage

src/audio.c

@@ -42,8 +42,12 @@
 #include <string.h>         // Required for strcmp()
 #include <stdio.h>          // Used for .WAV loading
 
-#include "utils.h"          // rRES data decompression utility function
+#if defined(AUDIO_STANDALONE)
+    #include <stdarg.h>     // Used for functions with variable number of parameters (TraceLog())
+#else
+    #include "utils.h"      // rRES data decompression utility function
                             // NOTE: Includes Android fopen function map
+#endif
 
 //#define STB_VORBIS_HEADER_ONLY
 #include "stb_vorbis.h"     // OGG loading functions 
@@ -81,6 +85,10 @@ typedef struct Music {
 
 } Music;
 
+#if defined(AUDIO_STANDALONE)
+typedef enum { INFO = 0, ERROR, WARNING, DEBUG, OTHER } TraceLogType;
+#endif
+
 //----------------------------------------------------------------------------------
 // Global Variables Definition
 //----------------------------------------------------------------------------------
@@ -99,6 +107,11 @@ static bool BufferMusicStream(ALuint buffer);   // Fill music buffers with data
 static void EmptyMusicStream(void);                 // Empty music buffers
 extern void UpdateMusicStream(void);                // Updates buffers (refill) for music streaming
 
+#if defined(AUDIO_STANDALONE)
+const char *GetExtension(const char *fileName);     // Get the extension for a filename
+void TraceLog(int msgType, const char *text, ...);  // Outputs a trace log message (INFO, ERROR, WARNING)
+#endif
+
 //----------------------------------------------------------------------------------
 // Module Functions Definition - Audio Device initialization and Closing
 //----------------------------------------------------------------------------------
@@ -279,6 +292,11 @@ Sound LoadSoundFromRES(const char *rresName, int resId)
 {
     // NOTE: rresName could be directly a char array with all the data!!! --> TODO
     Sound sound;
+
+#if defined(AUDIO_STANDALONE)
+    TraceLog(WARNING, "Sound loading from rRES resource file not supported on standalone mode");
+#else
+    
     bool found = false;
     
     char id[4];             // rRES file identifier
@@ -422,7 +440,7 @@ Sound LoadSoundFromRES(const char *rresName, int resId)
     }
 
     if (!found) TraceLog(WARNING, "[%s] Required resource id [%i] could not be found in the raylib resource file", rresName, resId);
-
+#endif
     return sound;
 }
 
@@ -783,6 +801,7 @@ static Wave LoadWAV(const char *fileName)
     if (wavFile == NULL)
     {
         TraceLog(WARNING, "[%s] WAV file could not be opened", fileName);
+        wave.data = NULL;
     }
     else
     {
@@ -852,6 +871,14 @@ static Wave LoadOGG(char *fileName)
     Wave wave;
 
     stb_vorbis *oggFile = stb_vorbis_open_filename(fileName, NULL, NULL);
+
+    if (oggFile == NULL)
+    {
+        TraceLog(WARNING, "[%s] OGG file could not be opened", fileName);
+        wave.data = NULL;
+    }
+    else
+    {
         stb_vorbis_info info = stb_vorbis_get_info(oggFile);
 
         wave.sampleRate = info.sample_rate;
@@ -886,6 +913,7 @@ static Wave LoadOGG(char *fileName)
         TraceLog(INFO, "[%s] OGG file loaded successfully (SampleRate: %i, BitRate: %i, Channels: %i)", fileName, wave.sampleRate, wave.bitsPerSample, wave.channels);
 
         stb_vorbis_close(oggFile);
+    }
 
     return wave;
 }
@@ -895,3 +923,46 @@ static void UnloadWave(Wave wave)
 {
     free(wave.data);
 }
+
+// Some required functions for audio standalone module version
+#if defined(AUDIO_STANDALONE)
+// Get the extension for a filename
+const char *GetExtension(const char *fileName)
+{
+    const char *dot = strrchr(fileName, '.');
+    if(!dot || dot == fileName) return "";
+    return (dot + 1);
+}
+
+// Outputs a trace log message (INFO, ERROR, WARNING)
+// NOTE: If a file has been init, output log is written there
+void TraceLog(int msgType, const char *text, ...)
+{
+    va_list args;
+    int traceDebugMsgs = 0;
+
+#ifdef DO_NOT_TRACE_DEBUG_MSGS
+    traceDebugMsgs = 0;
+#endif
+
+    switch(msgType)
+    {
+        case INFO: fprintf(stdout, "INFO: "); break;
+        case ERROR: fprintf(stdout, "ERROR: "); break;
+        case WARNING: fprintf(stdout, "WARNING: "); break;
+        case DEBUG: if (traceDebugMsgs) fprintf(stdout, "DEBUG: "); break;
+        default: break;
+    }
+
+    if ((msgType != DEBUG) || ((msgType == DEBUG) && (traceDebugMsgs)))
+    {
+        va_start(args, text);
+        vfprintf(stdout, text, args);
+        va_end(args);
+
+        fprintf(stdout, "\n");
+    }
+
+    if (msgType == ERROR) exit(1);      // If ERROR message, exit program
+}
+#endif

src/audio.h

@@ -95,7 +95,6 @@ void SetMusicVolume(float volume);                              // Set volume fo
 float GetMusicTimeLength(void);                                 // Get current music time length (in seconds)
 float GetMusicTimePlayed(void);                                 // Get current music time played (in seconds)
 
-
 #ifdef __cplusplus
 }
 #endif