Minor media3 navigator fixes (readium#447)

marshal1919 · Jan 16, 2024 · eb2f4e3 · eb2f4e3
1 parent cb9d2c0
commit eb2f4e3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/...ia/common/src/main/java/org/readium/navigator/media/common/DefaultMediaMetadataFactory.kt b/...ia/common/src/main/java/org/readium/navigator/media/common/DefaultMediaMetadataFactory.kt
@@ -45,7 +45,7 @@ internal class DefaultMediaMetadataFactory(
             // byte array will go cross processes and should be kept small
             publication.coverFitting(Size(400, 400))
                 ?.compress(Bitmap.CompressFormat.PNG, 80, byteStream)
-            byteStream.toByteArray()
+                ?.let { byteStream.toByteArray() }
         }
     }
 

diff --git a/...d/src/main/java/org/readium/r2/shared/publication/protection/FallbackContentProtection.kt b/...d/src/main/java/org/readium/r2/shared/publication/protection/FallbackContentProtection.kt
@@ -18,7 +18,7 @@ import org.readium.r2.shared.util.format.AdeptSpecification
 import org.readium.r2.shared.util.format.LcpSpecification
 
 /**
- * [ContentProtection] implementation used as a fallback by when detecting known DRMs
+ * [ContentProtection] implementation used as a fallback when detecting known DRMs
  * not supported by the app.
  */
 public class FallbackContentProtection : ContentProtection {

diff --git a/readium/streamer/src/main/java/org/readium/r2/streamer/parser/audio/AudioParser.kt b/readium/streamer/src/main/java/org/readium/r2/streamer/parser/audio/AudioParser.kt
@@ -63,7 +63,7 @@ public class AudioParser(
     private fun parseResourceAsset(
         asset: ResourceAsset
     ): Try<Publication.Builder, PublicationParser.ParseError> {
-        if (asset.format.conformsToAny(audioSpecifications)) {
+        if (!asset.format.conformsToAny(audioSpecifications)) {
             return Try.failure(PublicationParser.ParseError.FormatNotSupported())
         }