From aebc7da82b38ccb0f796ee50c610c5fc0aecc3f4 Mon Sep 17 00:00:00 2001 From: tonihei Date: Tue, 5 Dec 2017 06:57:27 -0800 Subject: [PATCH] Immediately release temp buffer memory in FakeRenderer. FakeRenderer only needs to allocate real memory because it extends BaseRenderer which uses the actual SampleStream implementation. Immediately release the memory after using it to prevent excessive memory usage when running fast simulations. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=177949628 --- .../com/google/android/exoplayer2/testutil/FakeRenderer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeRenderer.java b/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeRenderer.java index c4270eb9c4..75adcf9018 100644 --- a/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeRenderer.java +++ b/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeRenderer.java @@ -59,10 +59,10 @@ public class FakeRenderer extends BaseRenderer { @Override public void render(long positionUs, long elapsedRealtimeUs) throws ExoPlaybackException { if (!isEnded) { - buffer.clear(); // Verify the format matches the expected format. FormatHolder formatHolder = new FormatHolder(); int result = readSource(formatHolder, buffer, false); + buffer.data = null; if (result == C.RESULT_FORMAT_READ) { formatReadCount++; Assert.assertTrue(expectedFormats.contains(formatHolder.format));