returns when all requests for a source are removed, prevents locate() crash (#20)

akshar-bhatnagar · Akshar Bhatnagar · web-flow · commit b2494d1550e8 · 2022-05-23T19:43:11.000+05:30
Co-authored-by: Akshar Bhatnagar &lt;akshar@Akshars-MacBook-Pro.local&gt;
diff --git a/lib/src/main/java/com/otaliastudios/transcoder/internal/thumbnails/DefaultThumbnailsEngine.kt b/lib/src/main/java/com/otaliastudios/transcoder/internal/thumbnails/DefaultThumbnailsEngine.kt
@@ -290,6 +290,7 @@ class DefaultThumbnailsEngine(
             stubs.removeAll{
                 it.request.sourceId() == source
             }
+            return
         }
         if (stubs.firstOrNull()?.request?.sourceId() == source && positionUs == stubs.firstOrNull()?.positionUs) {
             return

Original file line number	Diff line number	Diff line change
`@@ -290,6 +290,7 @@ class DefaultThumbnailsEngine(`
`290`	`290`	`stubs.removeAll{`
`291`	`291`	`it.request.sourceId() == source`
`292`	`292`	`}`
	`293`	`+ return`
`293`	`294`	`}`
`294`	`295`	`if (stubs.firstOrNull()?.request?.sourceId() == source && positionUs == stubs.firstOrNull()?.positionUs) {`
`295`	`296`	`return`