From 2568ff73cbe0cbd92e192e8db7ec158c57792f34 Mon Sep 17 00:00:00 2001 From: kimvde Date: Mon, 18 Nov 2024 01:02:08 -0800 Subject: [PATCH] Remove unnecessary checkStateNotNull checks LongArrayQueue.remove() throws a NoSuchElementException if the queue is empty. PiperOrigin-RevId: 697531458 --- .../media3/exoplayer/video/VideoFrameRenderControl.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/VideoFrameRenderControl.java b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/VideoFrameRenderControl.java index 4ebd882eef..e759f71b37 100644 --- a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/VideoFrameRenderControl.java +++ b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/VideoFrameRenderControl.java @@ -17,7 +17,6 @@ package androidx.media3.exoplayer.video; import static androidx.media3.common.util.Assertions.checkArgument; import static androidx.media3.common.util.Assertions.checkNotNull; -import static androidx.media3.common.util.Assertions.checkStateNotNull; import androidx.annotation.Nullable; import androidx.media3.common.C; @@ -197,12 +196,12 @@ import androidx.media3.exoplayer.ExoPlaybackException; } private void dropFrame() { - checkStateNotNull(presentationTimestampsUs.remove()); + presentationTimestampsUs.remove(); frameRenderer.dropFrame(); } private void renderFrame(boolean shouldRenderImmediately) { - long presentationTimeUs = checkStateNotNull(presentationTimestampsUs.remove()); + long presentationTimeUs = presentationTimestampsUs.remove(); boolean videoSizeUpdated = maybeUpdateOutputVideoSize(presentationTimeUs); if (videoSizeUpdated) {