fix: reuse unloadSounds for unloading single player

android/src/main/cpp/AudioConstants.h

@@ -35,8 +35,4 @@ struct LoadSoundResult {
     std::optional<std::string> error;
 };
 
-struct UnloadSoundResult {
-    std::optional<std::string> error;
-};
-
 #endif //AUDIOPLAYBACK_AUDIOCONSTANTS_H

android/src/main/cpp/AudioEngine.cpp

@@ -173,17 +173,6 @@ LoadSoundResult AudioEngine::loadSound(int fd, int offset, int length) {
     return {.id = id, .error = std::nullopt};
 }
 
-UnloadSoundResult AudioEngine::unloadSound(const std::string &playerId) {
-    auto it = mPlayers.find(playerId);
-    if(it != mPlayers.end()) {
-        mPlayers.erase(it);
-    } else {
-        return {.error = "Audio file could not be unloaded because it is not found"};
-    }
-
-    return {.error = std::nullopt};
-}
-
 void AudioEngine::unloadSounds(const std::optional<std::vector<std::string>> &ids)  {
     if(ids.has_value()) {
         for (const auto & id: ids.value()) {

android/src/main/cpp/AudioEngine.h

@@ -25,7 +25,6 @@ class AudioEngine : public oboe::AudioStreamDataCallback{
     void seekSoundsTo(const std::vector<std::pair<std::string, double>>&);
     void setSoundsVolume(const std::vector<std::pair<std::string, double>>&);
     LoadSoundResult loadSound(int fd, int offset, int length);
-    UnloadSoundResult unloadSound(const std::string &playerId);
     void unloadSounds(const std::optional<std::vector<std::string>>&);
 
     oboe::DataCallbackResult onAudioReady(oboe::AudioStream *oboeStream, void *audioData, int32_t numFrames) override;

android/src/main/cpp/native-lib.cpp

@@ -77,6 +77,8 @@ std::vector<std::string> jniStringArrayToStringVector(JNIEnv* env, jobjectArray
         env->ReleaseStringUTFChars(javaString, chars);
         env->DeleteLocalRef(javaString);
     }
+
+    return cppVector;
 }
 
 extern "C" {
@@ -149,24 +151,6 @@ Java_com_audioplayback_AudioPlaybackModule_closeAudioStreamNative(JNIEnv *env, j
     return returnValue;
 }
 
-JNIEXPORT jobject JNICALL
-Java_com_audioplayback_AudioPlaybackModule_unloadSoundNative(JNIEnv *env, jobject instance, jstring playerId) {
-    auto result = audioEngine->unloadSound(jstringToStdString(env, playerId));
-
-    jclass structClass = env->FindClass("com/audioplayback/models/UnloadSoundResult");
-    jmethodID constructor = env->GetMethodID(structClass, "<init>", "(Ljava/lang/String;)V");
-
-    jstring jError = result.error.has_value() ? env->NewStringUTF(result.error->c_str()): nullptr;
-    jobject returnValue = env->NewObject(structClass, constructor, jError);
-
-    if(jError) {
-        env->DeleteLocalRef(jError);
-    }
-
-    return returnValue;
-}
-
-
 JNIEXPORT void JNICALL
 Java_com_audioplayback_AudioPlaybackModule_unloadSoundsNative(JNIEnv *env, jobject thiz,
                                                               jobjectArray ids) {

android/src/main/java/com/audioplayback/AudioPlaybackModule.kt

@@ -12,7 +12,6 @@ import com.audioplayback.models.LoadSoundResult
 import com.audioplayback.models.OpenAudioStreamResult
 import com.audioplayback.models.PauseAudioStreamResult
 import com.audioplayback.models.SetupAudioStreamResult
-import com.audioplayback.models.UnloadSoundResult
 import com.facebook.react.bridge.Arguments
 import com.facebook.react.bridge.WritableMap
 import kotlinx.coroutines.CoroutineScope
@@ -88,12 +87,9 @@ class AudioPlaybackModule internal constructor(context: ReactApplicationContext)
   }
 
 
-  @ReactMethod(isBlockingSynchronousMethod = true)
-  override fun unloadSound(id: String): WritableMap {
-    val result = unloadSoundNative(id)
-    val map = Arguments.createMap()
-    result.error?.let { map.putString("error", it) } ?: map.putNull("error")
-    return map
+  @ReactMethod
+  override fun unloadSound(id: String) {
+    unloadSoundsNative(arrayOf(id))
   }
 
   @ReactMethod
@@ -198,7 +194,6 @@ class AudioPlaybackModule internal constructor(context: ReactApplicationContext)
   private external fun seekSoundsToNative(ids: Array<String>, values: DoubleArray)
   private external fun setSoundsVolumeNative(ids: Array<String>, values: DoubleArray)
   private external fun loadSoundNative(fd: Int, fileLength: Int, fileOffset: Int): LoadSoundResult
-  private external fun unloadSoundNative(playerId: String): UnloadSoundResult
   private external fun unloadSoundsNative(ids: Array<String>?)
 
   // Example method

android/src/main/java/com/audioplayback/models/AudioPlaybackMethodsResults.kt

@@ -5,4 +5,3 @@ data class OpenAudioStreamResult(val error: String?)
 data class PauseAudioStreamResult(val error: String?)
 data class CloseAudioStreamResult(val error: String?)
 data class LoadSoundResult(val error: String?, val id: String?)
-data class UnloadSoundResult(val error: String?)

android/src/oldarch/AudioPlaybackSpec.kt

@@ -26,7 +26,7 @@ abstract class AudioPlaybackSpec internal constructor(context: ReactApplicationC
 
   abstract fun setSoundsVolume(arg: ReadableArray)
 
-  abstract fun unloadSound(id: String): WritableMap
+  abstract fun unloadSound(id: String)
 
   abstract fun loadSound(uri: String, promise: Promise)
 }

example/ios/Podfile.lock

@@ -1242,7 +1242,7 @@ PODS:
     - ReactCommon/turbomodule/bridging
     - ReactCommon/turbomodule/core
     - Yoga
-  - react-native-audio-playback (0.2.0-alpha.4):
+  - react-native-audio-playback (1.0.0):
     - DoubleConversion
     - glog
     - hermes-engine
@@ -1818,7 +1818,7 @@ SPEC CHECKSUMS:
   React-logger: addd140841248966c2547eb94836399cc1061f4d
   React-Mapbuffer: 029b5332e78af8c67c4b5e65edfc717068b8eac1
   React-microtasksnativemodule: f30949ee318ba90b9668de1e325b98838b9a4da2
-  react-native-audio-playback: 9928a4f29998a3d98f1ee63e8054bf23c97ff86b
+  react-native-audio-playback: 2e227d97930550dc443ae76be9a07b25f51c469e
   react-native-slider: d1a9121980fc81678c6d30b82f312c778fba563c
   React-nativeconfig: 470fce6d871c02dc5eff250a362d56391b7f52d6
   React-NativeModulesApple: 1586448c61a7c2bd4040cc03ccde66a72037e77e

src/NativeAudioPlayback.ts

@@ -13,7 +13,7 @@ export interface Spec extends TurboModule {
   playSounds: (arg: Array<[string, boolean]>) => void;
   seekSoundsTo: (arg: Array<[string, number]>) => void;
   setSoundsVolume: (arg: Array<[string, number]>) => void;
-  unloadSound: (id: string) => { error: string | null };
+  unloadSound: (id: string) => void;
   loadSound: (
     uri: string
   ) => Promise<{ id: string | null; error: string | null }>;

src/module.ts

@@ -93,8 +93,5 @@ export async function loadSound(requiredAsset: number): Promise<string> {
 }
 
 export function unloadSound(playerId: string) {
-  const res = AudioPlayback.unloadSound(playerId);
-  if (res.error) {
-    throw new Error(res.error);
-  }
+  AudioPlayback.unloadSound(playerId);
 }