Removes Pulse Audio. Config values will be migrated to sdl audio back…

… end. (#356) * Removes Pulse Audio. Config values will be migrated to sdl audio backend. #311 * Removes PulseAudio.h from classes.h
Kenix3 · Oct 10, 2023 · 8fb8711 · 8fb8711
1 parent 317edd7
commit 8fb8711
Show file tree

Hide file tree

Showing 7 changed files with 5 additions and 241 deletions.
diff --git a/include/libultraship/classes.h b/include/libultraship/classes.h
@@ -31,9 +31,7 @@
 #include "utils/binarytools/BinaryWriter.h"
 #include "audio/Audio.h"
 #include "audio/AudioPlayer.h"
-#if defined(__linux__) || defined(__BSD__)
-#include "audio/PulseAudioPlayer.h"
-#elif defined(_WIN32)
+#if defined(_WIN32)
 #include "audio/WasapiAudioPlayer.h"
 #endif
 #include "audio/SDLAudioPlayer.h"

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -29,16 +29,11 @@ set(Source_Files__Audio
 	${CMAKE_CURRENT_SOURCE_DIR}/audio/AudioPlayer.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/audio/SDLAudioPlayer.h
     ${CMAKE_CURRENT_SOURCE_DIR}/audio/SDLAudioPlayer.cpp
-    ${CMAKE_CURRENT_SOURCE_DIR}/audio/PulseAudioPlayer.h
-    ${CMAKE_CURRENT_SOURCE_DIR}/audio/PulseAudioPlayer.cpp
 )
 
 if (CMAKE_SYSTEM_NAME STREQUAL "Windows")
     list(APPEND Source_Files__Audio ${CMAKE_CURRENT_SOURCE_DIR}/audio/WasapiAudioPlayer.h)
     list(APPEND Source_Files__Audio ${CMAKE_CURRENT_SOURCE_DIR}/audio/WasapiAudioPlayer.cpp)
-elseif (CMAKE_SYSTEM_NAME STREQUAL "Linux")
-    list(APPEND Source_Files__Audio ${CMAKE_CURRENT_SOURCE_DIR}/audio/PulseAudioPlayer.h)
-    list(APPEND Source_Files__Audio ${CMAKE_CURRENT_SOURCE_DIR}/audio/PulseAudioPlayer.cpp)
 endif()
 
 source_group("audio" FILES ${Source_Files__Audio})
@@ -127,7 +122,7 @@ set(Source_Files__Log
     ${CMAKE_CURRENT_SOURCE_DIR}/log/luslog.cpp
 )
 
-source_group("Log" FILES ${Source_Files__Log})
+source_group("log" FILES ${Source_Files__Log})
 
 target_sources(libultraship PRIVATE ${Source_Files__Log} ${Source_Files__Log__SPDLog})
 

diff --git a/src/audio/Audio.cpp b/src/audio/Audio.cpp
@@ -16,11 +16,6 @@ void Audio::InitAudioPlayer() {
         case AudioBackend::WASAPI:
             mAudioPlayer = std::make_shared<WasapiAudioPlayer>();
             break;
-#endif
-#if defined(__linux)
-        case AudioBackend::PULSE:
-            mAudioPlayer = std::make_shared<PulseAudioPlayer>();
-            break;
 #endif
         default:
             mAudioPlayer = std::make_shared<SDLAudioPlayer>();

diff --git a/src/audio/AudioPlayer.h b/src/audio/AudioPlayer.h
@@ -31,8 +31,6 @@ class AudioPlayer {
 
 #ifdef _WIN32
 #include "WasapiAudioPlayer.h"
-#elif defined(__linux)
-#include "PulseAudioPlayer.h"
 #endif
 
 #include "SDLAudioPlayer.h"
diff --git a/src/audio/PulseAudioPlayer.cpp b/src/audio/PulseAudioPlayer.cpp
diff --git a/src/audio/PulseAudioPlayer.h b/src/audio/PulseAudioPlayer.h
diff --git a/src/config/Config.cpp b/src/config/Config.cpp
@@ -178,8 +178,10 @@ AudioBackend Config::GetAudioBackend() {
         return AudioBackend::WASAPI;
     }
 
+    // Migrate pulse player in config to sdl
     if (backendName == "pulse") {
-        return AudioBackend::PULSE;
+        SetAudioBackend(AudioBackend::SDL);
+        return AudioBackend::SDL;
     }
 
     if (backendName == "sdl") {
@@ -190,8 +192,6 @@ AudioBackend Config::GetAudioBackend() {
                  backendName);
 #ifdef _WIN32
     return AudioBackend::WASAPI;
-#elif defined(__linux)
-    return AudioBackend::PULSE;
 #endif
 
     return AudioBackend::SDL;
@@ -202,9 +202,6 @@ void Config::SetAudioBackend(AudioBackend backend) {
         case AudioBackend::WASAPI:
             SetString("Window.AudioBackend", "wasapi");
             break;
-        case AudioBackend::PULSE:
-            SetString("Window.AudioBackend", "pulse");
-            break;
         case AudioBackend::SDL:
             SetString("Window.AudioBackend", "sdl");
             break;