From 0afb13014b370be8a7367053beadfca2cf3a8f89 Mon Sep 17 00:00:00 2001 From: kwiberg Date: Wed, 24 May 2017 04:09:52 -0700 Subject: [PATCH] AudioEncoderPcm16B: Number of bits/sample is 16, not 2 Clearly, this was a case of bit/byte confusion. BUG=none Review-Url: https://codereview.webrtc.org/2904883002 Cr-Commit-Position: refs/heads/master@{#18254} --- .../modules/audio_coding/codecs/pcm16b/audio_encoder_pcm16b.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/webrtc/modules/audio_coding/codecs/pcm16b/audio_encoder_pcm16b.cc b/webrtc/modules/audio_coding/codecs/pcm16b/audio_encoder_pcm16b.cc index 442ea662dd..f57cc7ebdc 100644 --- a/webrtc/modules/audio_coding/codecs/pcm16b/audio_encoder_pcm16b.cc +++ b/webrtc/modules/audio_coding/codecs/pcm16b/audio_encoder_pcm16b.cc @@ -84,9 +84,10 @@ rtc::Optional AudioEncoderPcm16B::QueryAudioEncoder( format.num_channels >= 1) { Config config = CreateConfig(0, format); if (config.IsOk()) { + constexpr int bits_per_sample = 16; return rtc::Optional( {config.sample_rate_hz, config.num_channels, - config.sample_rate_hz * 2 * + config.sample_rate_hz * bits_per_sample * rtc::dchecked_cast(config.num_channels)}); } }