Release BufferFilter's when renderer is released (#200)

linkedin · Jul 1, 2022 · 409a707 · 409a707
1 parent c74f270
commit 409a707
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/litr-filters/src/main/java/com/linkedin/android/litr/filter/audio/VolumeFilter.kt b/litr-filters/src/main/java/com/linkedin/android/litr/filter/audio/VolumeFilter.kt
@@ -39,4 +39,6 @@ class VolumeFilter(@FloatRange(from = 0.0) private val volume: Double) : BufferF
             }
         }
     }
+
+    override fun release() {}
 }
diff --git a/litr/src/main/java/com/linkedin/android/litr/filter/BufferFilter.kt b/litr/src/main/java/com/linkedin/android/litr/filter/BufferFilter.kt
@@ -21,4 +21,9 @@ interface BufferFilter {
      * sample rate and channel count.
      */
     fun apply(frame: Frame)
+
+    /**
+     * Release the filter.
+     */
+    fun release()
 }
diff --git a/litr/src/main/java/com/linkedin/android/litr/render/AudioRenderer.kt b/litr/src/main/java/com/linkedin/android/litr/render/AudioRenderer.kt
@@ -111,6 +111,7 @@ class AudioRenderer @JvmOverloads constructor(
         released.set(true)
         audioProcessor?.release()
         bufferPool.clear()
+        filters.forEach { it.release() }
     }
 
     override fun hasFilters(): Boolean {