From 4983d4f33957686be2f3c4eea09ce8914f2c2150 Mon Sep 17 00:00:00 2001 From: samrobinson Date: Tue, 11 Jul 2023 16:12:52 +0100 Subject: [PATCH] Switch argument order passed to scaleLargeTimestamp. durationUs is almost always going to be a larger number than the sample rate, so pass as the main value, rather than the multiplier. PiperOrigin-RevId: 547193927 --- .../androidx/media3/transformer/SilentAudioGenerator.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/SilentAudioGenerator.java b/libraries/transformer/src/main/java/androidx/media3/transformer/SilentAudioGenerator.java index 43b74ac81a..8837fc0e1d 100644 --- a/libraries/transformer/src/main/java/androidx/media3/transformer/SilentAudioGenerator.java +++ b/libraries/transformer/src/main/java/androidx/media3/transformer/SilentAudioGenerator.java @@ -53,8 +53,8 @@ import java.util.concurrent.atomic.AtomicLong; // overflow-safe multiplication & division. long outputFrameCount = Util.scaleLargeTimestamp( - /* timestamp= */ audioFormat.sampleRate, - /* multiplier= */ durationUs, + /* timestamp= */ durationUs, + /* multiplier= */ audioFormat.sampleRate, /* divisor= */ C.MICROS_PER_SECOND); remainingBytesToOutput.addAndGet(audioFormat.bytesPerFrame * outputFrameCount);