Switch acm_receiver over to using base/logging.h
BUG=
R=henrik.lundin@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/57439004
Cr-Commit-Position: refs/heads/master@{#9298}
diff --git a/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc b/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc
index 2b562d2..f9cf89a 100644
--- a/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc
+++ b/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc
@@ -16,6 +16,7 @@
#include <vector>
#include "webrtc/base/format_macros.h"
+#include "webrtc/base/logging.h"
#include "webrtc/common_audio/signal_processing/include/signal_processing_library.h"
#include "webrtc/common_types.h"
#include "webrtc/modules/audio_coding/codecs/audio_decoder.h"
@@ -26,7 +27,6 @@
#include "webrtc/modules/audio_coding/neteq/interface/neteq.h"
#include "webrtc/system_wrappers/interface/clock.h"
#include "webrtc/system_wrappers/interface/critical_section_wrapper.h"
-#include "webrtc/system_wrappers/interface/logging.h"
#include "webrtc/system_wrappers/interface/tick_util.h"
#include "webrtc/system_wrappers/interface/trace.h"
@@ -99,7 +99,7 @@
// not active. However, if post-decoding VAD has been active then
// disabled, we might be here for couple of frames.
audio_frame->speech_type_ = AudioFrame::kNormalSpeech;
- LOG_F(LS_WARNING) << "Post-decoding VAD is disabled but output is "
+ LOG(WARNING) << "Post-decoding VAD is disabled but output is "
<< "labeled VAD-passive";
break;
}
@@ -156,7 +156,7 @@
int AcmReceiver::SetMinimumDelay(int delay_ms) {
if (neteq_->SetMinimumDelay(delay_ms))
return 0;
- LOG_FERR1(LS_ERROR, "AcmReceiver::SetExtraDelay", delay_ms);
+ LOG(LERROR) << "AcmReceiver::SetExtraDelay " << delay_ms;
return -1;
}
@@ -199,7 +199,7 @@
int AcmReceiver::SetMaximumDelay(int delay_ms) {
if (neteq_->SetMaximumDelay(delay_ms))
return 0;
- LOG_FERR1(LS_ERROR, "AcmReceiver::SetExtraDelay", delay_ms);
+ LOG(LERROR) << "AcmReceiver::SetExtraDelay " << delay_ms;
return -1;
}
@@ -334,9 +334,9 @@
if (neteq_->InsertPacket(rtp_header, incoming_payload, length_payload,
receive_timestamp) < 0) {
- LOG_FERR1(LS_ERROR, "AcmReceiver::InsertPacket",
- static_cast<int>(header->payloadType))
- << " Failed to insert packet";
+ LOG(LERROR) << "AcmReceiver::InsertPacket "
+ << static_cast<int>(header->payloadType)
+ << " Failed to insert packet";
return -1;
}
return 0;
@@ -379,7 +379,7 @@
&samples_per_channel,
&num_channels,
&type) != NetEq::kOK) {
- LOG_FERR0(LS_ERROR, "AcmReceiver::GetAudio") << "NetEq Failed.";
+ LOG(LERROR) << "AcmReceiver::GetAudio - NetEq Failed.";
return -1;
}
@@ -411,8 +411,8 @@
AudioFrame::kMaxDataSizeSamples,
temp_output);
if (samples_per_channel < 0) {
- LOG_FERR0(LS_ERROR, "AcmReceiver::GetAudio")
- << "Resampling last_audio_buffer_ failed.";
+ LOG(LERROR) << "AcmReceiver::GetAudio - "
+ "Resampling last_audio_buffer_ failed.";
return -1;
}
}
@@ -430,8 +430,7 @@
AudioFrame::kMaxDataSizeSamples,
audio_frame->data_);
if (samples_per_channel < 0) {
- LOG_FERR0(LS_ERROR, "AcmReceiver::GetAudio")
- << "Resampling audio_buffer_ failed.";
+ LOG(LERROR) << "AcmReceiver::GetAudio - Resampling audio_buffer_ failed.";
return -1;
}
resampled_last_output_frame_ = true;
@@ -501,8 +500,7 @@
// Changing codec or number of channels. First unregister the old codec,
// then register the new one.
if (neteq_->RemovePayloadType(payload_type) != NetEq::kOK) {
- LOG_F(LS_ERROR) << "Cannot remove payload "
- << static_cast<int>(payload_type);
+ LOG(LERROR) << "Cannot remove payload " << static_cast<int>(payload_type);
return -1;
}
@@ -517,8 +515,9 @@
payload_type, sample_rate_hz);
}
if (ret_val != NetEq::kOK) {
- LOG_FERR3(LS_ERROR, "AcmReceiver::AddCodec", acm_codec_id,
- static_cast<int>(payload_type), channels);
+ LOG(LERROR) << "AcmReceiver::AddCodec " << acm_codec_id
+ << static_cast<int>(payload_type)
+ << " channels: " << channels;
return -1;
}
@@ -576,8 +575,7 @@
return 0;
}
if (neteq_->RemovePayloadType(payload_type) != NetEq::kOK) {
- LOG_FERR1(LS_ERROR, "AcmReceiver::RemoveCodec",
- static_cast<int>(payload_type));
+ LOG(LERROR) << "AcmReceiver::RemoveCodec" << static_cast<int>(payload_type);
return -1;
}
if (last_audio_decoder_ == &it->second)
@@ -615,7 +613,7 @@
return decoder.payload_type;
}
}
- LOG_F(LS_WARNING) << "RED is not registered.";
+ LOG(WARNING) << "RED is not registered.";
return -1;
}
@@ -681,8 +679,8 @@
CriticalSectionScoped lock(crit_sect_.get());
auto it = decoders_.find(payload_type);
if (it == decoders_.end()) {
- LOG_FERR1(LS_ERROR, "AcmReceiver::DecoderByPayloadType",
- static_cast<int>(payload_type));
+ LOG(LERROR) << "AcmReceiver::DecoderByPayloadType "
+ << static_cast<int>(payload_type);
return -1;
}
const Decoder& decoder = it->second;