From 6c55f25bfd11b64b29990e94932c323fc18576fb Mon Sep 17 00:00:00 2001 From: christosts Date: Wed, 5 Apr 2023 14:52:15 +0100 Subject: [PATCH] Small clean-up on BaseRenderer PiperOrigin-RevId: 522039856 --- .../androidx/media3/exoplayer/BaseRenderer.java | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/BaseRenderer.java b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/BaseRenderer.java index 40c717378b..b11e946e90 100644 --- a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/BaseRenderer.java +++ b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/BaseRenderer.java @@ -56,7 +56,7 @@ public abstract class BaseRenderer implements Renderer, RendererCapabilities { @GuardedBy("lock") @Nullable - protected RendererCapabilities.Listener rendererCapabilitiesListener; + private RendererCapabilities.Listener rendererCapabilitiesListener; /** * @param trackType The track type that the renderer handles. One of the {@link C} {@code @@ -231,13 +231,6 @@ public abstract class BaseRenderer implements Renderer, RendererCapabilities { } } - @Nullable - private Listener getListener() { - synchronized (lock) { - return this.rendererCapabilitiesListener; - } - } - // PlayerMessage.Target implementation. @Override @@ -517,7 +510,10 @@ public abstract class BaseRenderer implements Renderer, RendererCapabilities { /** Called when the renderer capabilities are changed. */ protected final void onRendererCapabilitiesChanged() { - @Nullable RendererCapabilities.Listener listener = getListener(); + @Nullable RendererCapabilities.Listener listener; + synchronized (lock) { + listener = rendererCapabilitiesListener; + } if (listener != null) { listener.onRendererCapabilitiesChanged(this); }