Skip to content

Commit

Permalink
Update nits
Browse files Browse the repository at this point in the history
  • Loading branch information
cedricxperi committed Sep 18, 2023
1 parent fb92386 commit 97e2bb0
Showing 1 changed file with 4 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,7 @@ public class DefaultAudioTrackBufferSizeProvider
*/
private static final int DTSHD_BUFFER_MULTIPLICATION_FACTOR = 4;

/**
* A builder to create {@link DefaultAudioTrackBufferSizeProvider} instances.
*/
/** A builder to create {@link DefaultAudioTrackBufferSizeProvider} instances. */

public static class Builder {

Expand Down Expand Up @@ -156,7 +154,7 @@ public Builder setAc3BufferMultiplicationFactor(int ac3BufferMultiplicationFacto
/**
* Sets the multiplication factor to apply to the passthrough buffer for DTS-HD (DTS Express)
* to avoid underruns on some devices (e.g., Xiaomi A2 TV). Default is
* {@value #DTSHD_BUFFER_MULTIPLICATION_FACTOR}.
* {@link #DTSHD_BUFFER_MULTIPLICATION_FACTOR}.
*/
@CanIgnoreReturnValue
public Builder setDtshdBufferMultiplicationFactor(int dtshdBufferMultiplicationFactor) {
Expand Down Expand Up @@ -259,8 +257,8 @@ protected int getPassthroughBufferSizeInBytes(@C.Encoding int encoding, int bitr
if (encoding == C.ENCODING_AC3) {
bufferSizeUs *= ac3BufferMultiplicationFactor;
} else if (encoding == C.ENCODING_DTS_HD) {
// DTSHD (DTS Express) for streaming uses a frame size (number of audio samples per channel
// per frame of 4096. This requires a higher multiple for the buffersize computation.
// DTS-HD (DTS Express) for streaming uses a frame size (number of audio samples per channel
// per frame) of 4096. This requires a higher multiple for the buffersize computation.
// Otherwise, there will be buffer underflow during DASH playback.
bufferSizeUs *= dtshdBufferMultiplicationFactor;
}
Expand Down

0 comments on commit 97e2bb0

Please sign in to comment.