nick/amethyst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Adds note to make sure we know the callback of a prepare video runs in the main thread.

Browse Source
This commit is contained in:
Vitor Pamplona
2023-09-26 10:56:28 -04:00
parent e3df94338f
commit 1387a0f62b
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/java/com/vitorpamplona/amethyst/ui/components/VideoView.kt
+4
View File
@@ -299,6 +299,8 @@ fun GetVideoController(
nostrUriCallback, nostrUriCallback,
context context
) { ) {
// REQUIRED TO BE RUN IN THE MAIN THREAD
// checks again because of race conditions. // checks again because of race conditions.
if (controller.value == null) { // still prone to race conditions. if (controller.value == null) { // still prone to race conditions.
controller.value = it controller.value = it
@@ -382,6 +384,8 @@ fun GetVideoController(
nostrUriCallback, nostrUriCallback,
context context
) { ) {
// REQUIRED TO BE RUN IN THE MAIN THREAD
// checks again to make sure no other thread has created a controller. // checks again to make sure no other thread has created a controller.
if (controller.value == null) { if (controller.value == null) {
controller.value = it controller.value = it