Rebase Audio Rework

Author: Raltyro

source/funkin/Conductor.hx

@@ -538,7 +538,7 @@ class Conductor
     if (soundToCheck == null) soundToCheck = FlxG.sound.music;
 
     @:privateAccess
-    this.songPosition = soundToCheck._channel.position;
+    this.songPosition = soundToCheck.getActualTime();
     return this.songPosition;
   }
 

source/funkin/FunkinMemory.hx

@@ -21,10 +21,6 @@ class FunkinMemory
   static var currentCachedTextures:Map<String, FlxGraphic> = [];
   static var previousCachedTextures:Map<String, FlxGraphic> = [];
 
-  static var permanentCachedSounds:Map<String, Sound> = [];
-  static var currentCachedSounds:Map<String, Sound> = [];
-  static var previousCachedSounds:Map<String, Sound> = [];
-
   static var purgeFilter:Array<String> = ["/week", "/characters", "/charSelect", "/results"];
 
   /**
@@ -102,8 +98,6 @@ class FunkinMemory
   {
     preparePurgeTextureCache();
     purgeTextureCache();
-    preparePurgeSoundCache();
-    purgeSoundCache();
     #if (cpp || neko || hl)
     if (callGarbageCollector) funkin.util.MemoryUtil.collect(true);
     #end
@@ -331,21 +325,7 @@ class FunkinMemory
    */
   public static function cacheSound(key:String):Void
   {
-    if (currentCachedSounds.exists(key)) return;
-
-    if (previousCachedSounds.exists(key))
-    {
-      // Move the texture from the previous cache to the current cache.
-      var sound:Null<Sound> = previousCachedSounds.get(key);
-      previousCachedSounds.remove(key);
-      if (sound != null) currentCachedSounds.set(key, sound);
-      return;
-    }
-
-    var sound:Null<Sound> = Assets.getSound(key, true);
-    if (sound == null) return;
-    else
-      currentCachedSounds.set(key, sound);
+    flixel.sound.FlxSoundData.fromAssetKey(key);
   }
 
   /**
@@ -354,64 +334,8 @@ class FunkinMemory
    */
   public static function permanentCacheSound(key:String):Void
   {
-    if (permanentCachedSounds.exists(key)) return;
-
-    var sound:Null<Sound> = Assets.getSound(key, true);
-    if (sound == null) return;
-    else
-      permanentCachedSounds.set(key, sound);
-
-    if (sound != null) currentCachedSounds.set(key, sound);
-  }
-
-  /**
-   * Prepares the cache for purging unused sounds.
-   */
-  public static function preparePurgeSoundCache():Void
-  {
-    previousCachedSounds = currentCachedSounds.copy();
-
-    for (key in previousCachedSounds.keys())
-    {
-      if (permanentCachedSounds.exists(key))
-      {
-        previousCachedSounds.remove(key);
-      }
-    }
-
-    currentCachedSounds = permanentCachedSounds.copy();
-  }
-
-  /**
-   * Purges unused sounds from the cache.
-   */
-  public static inline function purgeSoundCache():Void
-  {
-    for (key in previousCachedSounds.keys())
-    {
-      if (permanentCachedSounds.exists(key))
-      {
-        previousCachedSounds.remove(key);
-        continue;
-      }
-
-      var sound:Null<Sound> = previousCachedSounds.get(key);
-      if (sound != null)
-      {
-        Assets.cache.removeSound(key);
-        previousCachedSounds.remove(key);
-      }
-    }
-    Assets.cache.clear("songs");
-    Assets.cache.clear("music");
-    // Felt lazy.
-    var key = Paths.music("freakyMenu/freakyMenu");
-    var sound:Null<Sound> = Assets.getSound(key, true);
-    if (sound != null)
-    {
-      permanentCachedSounds.set(key, sound);
-      currentCachedSounds.set(key, sound);
-    }
+    var soundData = flixel.sound.FlxSoundData.fromAssetKey(key);
+    if (soundData != null) soundData.persist = true;
   }
 
   ///// MISC /////
@@ -445,9 +369,6 @@ class FunkinMemory
       if (currentCachedTextures.exists(key)) currentCachedTextures.remove(key);
       Assets.cache.clear(key);
     }
-
-    preparePurgeSoundCache();
-    purgeSoundCache();
   }
 
   /**

source/funkin/Preferences.hx

@@ -416,6 +416,32 @@ class Preferences
     return value;
   }
 
+  #if desktop
+  /**
+   * What audio device should it playback sounds to.
+   * @default Default
+   */
+  public static var audioDevice(get, set):String;
+
+  static function get_audioDevice():String
+  {
+    return Save?.instance?.options?.audioDevice ?? "Default";
+  }
+
+  static function set_audioDevice(value:String):String
+  {
+    if (value == Save.instance.options.audioDevice) return value;
+
+    var save:Save = Save.instance;
+    FlxG.sound.automaticDefaultDevice = value == "Default" || (FlxG.sound.deviceName = value) != value;
+
+    save.options.audioDevice = FlxG.sound.automaticDefaultDevice ? "Default" : value;
+    Save.system.flush();
+
+    return value;
+  }
+  #end
+
   /**
    * If enabled, the game will hide the mouse when taking a screenshot.
    * @default `true`
@@ -485,6 +511,20 @@ class Preferences
     setDebugDisplayMode(Preferences.debugDisplay);
     setDebugDisplayBGOpacity(Preferences.debugDisplayBGOpacity / 100);
 
+    #if desktop
+    // Apply audio device preference, if failed, fallback to Default.
+    FlxG.sound.deviceName = Preferences.audioDevice;
+    if (FlxG.sound.deviceName == Preferences.audioDevice)
+    {
+      FlxG.sound.automaticDefaultDevice = false;
+    }
+    else
+    {
+      Preferences.audioDevice = "Default";
+      FlxG.sound.automaticDefaultDevice = true;
+    }
+    #end
+
     #if web
     toggleFramerateCap(Preferences.unlockedFramerate);
     #end

source/funkin/audio/FlxStreamSound.hx

@@ -1,12 +1,11 @@
 package funkin.audio;
 
-import flash.media.Sound;
-import flixel.sound.FlxSound;
-import flixel.system.FlxAssets.FlxSoundAsset;
+import lime.media.AudioBuffer;
 import openfl.Assets;
-#if (openfl >= "8.0.0")
+import openfl.media.Sound;
 import openfl.utils.AssetType;
-#end
+import flixel.sound.FlxSound;
+import flixel.system.FlxAssets.FlxSoundAsset;
 
 /**
  * a FlxSound that just overrides loadEmbedded to allow for "streamed" sounds to load with better performance!
@@ -19,29 +18,16 @@ class FlxStreamSound extends FlxSound
     super();
   }
 
-  override public function loadEmbedded(EmbeddedSound:Null<FlxSoundAsset>, Looped:Bool = false, AutoDestroy:Bool = false, ?OnComplete:Void->Void):FlxSound
+  override public function loadEmbedded(asset:FlxSoundAsset, ?looped:Bool, ?loopTime:Float, ?endTime:Float, autoDestroy = false, ?onComplete:Void->Void):FlxStreamSound
   {
-    if (EmbeddedSound == null) return this;
-
-    cleanup(true);
-
-    if ((EmbeddedSound is Sound))
-    {
-      _sound = EmbeddedSound;
-    }
-    else if ((EmbeddedSound is Class))
+    if ((asset is String))
     {
-      _sound = Type.createInstance(EmbeddedSound, []);
+      super.loadStreamed(asset, looped, loopTime, endTime, autoDestroy, onComplete);
     }
-    else if ((EmbeddedSound is String))
+    else
     {
-      if (Assets.exists(EmbeddedSound, AssetType.SOUND)
-        || Assets.exists(EmbeddedSound, AssetType.MUSIC)) _sound = Assets.getMusic(EmbeddedSound);
-      else
-        FlxG.log.error('Could not find a Sound asset with an ID of \'$EmbeddedSound\'.');
+      super.loadEmbedded(asset, looped, loopTime, endTime, autoDestroy, onComplete);
     }
-
-    // NOTE: can't pull ID3 info from embedded sound currently
-    return init(Looped, AutoDestroy, OnComplete);
+    return this;
   }
 }