Cleaning up r7562-7567.
Wrongly used git svn dcommit for committing a CL. Then two reverts were applied. Still something needs to be cleaned. BUG= TBR=kjellander@webrtc.org Review URL: https://webrtc-codereview.appspot.com/23299004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@7568 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
parent
879fac81d1
commit
8219529b98
@ -110,26 +110,13 @@ static const int kDefaultAudioDeviceId = 0;
|
||||
|
||||
static const char kIsacCodecName[] = "ISAC";
|
||||
static const char kL16CodecName[] = "L16";
|
||||
|
||||
// Codec parameters for Opus.
|
||||
static const int kOpusMonoBitrate = 32000;
|
||||
// Parameter used for NACK.
|
||||
// This value is equivalent to 5 seconds of audio data at 20 ms per packet.
|
||||
static const int kNackMaxPackets = 250;
|
||||
|
||||
// Codec parameters for Opus.
|
||||
static const int kOpusStereoBitrate = 64000;
|
||||
// draft-spittka-payload-rtp-opus-03
|
||||
|
||||
// Recommended bitrates:
|
||||
// 8-12 kb/s for NB speech,
|
||||
// 16-20 kb/s for WB speech,
|
||||
// 28-40 kb/s for FB speech,
|
||||
// 48-64 kb/s for FB mono music, and
|
||||
// 64-128 kb/s for FB stereo music.
|
||||
// The current implementation applies the following values to mono signals,
|
||||
// and multiplies them by 2 for stereo.
|
||||
static const int kOpusBitrateNb = 12000;
|
||||
static const int kOpusBitrateWb = 20000;
|
||||
static const int kOpusBitrateFb = 32000;
|
||||
|
||||
// Opus bitrate should be in the range between 6000 and 510000.
|
||||
static const int kOpusMinBitrate = 6000;
|
||||
static const int kOpusMaxBitrate = 510000;
|
||||
@ -430,17 +417,8 @@ static int GetOpusBitrate(const AudioCodec& codec, int max_playback_rate) {
|
||||
use_param = false;
|
||||
}
|
||||
if (bitrate <= 0) {
|
||||
if (max_playback_rate <= 8000) {
|
||||
bitrate = kOpusBitrateNb;
|
||||
} else if (max_playback_rate <= 16000) {
|
||||
bitrate = kOpusBitrateWb;
|
||||
} else {
|
||||
bitrate = kOpusBitrateFb;
|
||||
}
|
||||
|
||||
if (IsOpusStereoEnabled(codec)) {
|
||||
bitrate *= 2;
|
||||
}
|
||||
bitrate = IsOpusStereoEnabled(codec) ? kOpusStereoBitrate :
|
||||
kOpusMonoBitrate;
|
||||
} else if (bitrate < kOpusMinBitrate || bitrate > kOpusMaxBitrate) {
|
||||
bitrate = (bitrate < kOpusMinBitrate) ? kOpusMinBitrate : kOpusMaxBitrate;
|
||||
std::string rate_source =
|
||||
@ -481,6 +459,9 @@ static void GetOpusConfig(const AudioCodec& codec, webrtc::CodecInst* voe_codec,
|
||||
// the bitrate is not specified, i.e. is <= zero, we set it to the
|
||||
// appropriate default value for mono or stereo Opus.
|
||||
|
||||
// TODO(minyue): The determination of bit rate might take the maximum playback
|
||||
// rate into account.
|
||||
|
||||
voe_codec->channels = IsOpusStereoEnabled(codec) ? 2 : 1;
|
||||
voe_codec->rate = GetOpusBitrate(codec, *max_playback_rate);
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user