admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Implement RTC_DLOG macro and use in a few places to test."

Browse Source 
This reverts commit b18e868aa4375351cacd79efeb519b123878c39a.

Reason for revert: Breaks Chromium

Original change's description:
> Implement RTC_DLOG macro and use in a few places to test.
> 
> Bug: webrtc:8529
> Change-Id: I31a5a4ec873088b1831dfa31a496467efff89cc1
> Reviewed-on: https://webrtc-review.googlesource.com/23041
> Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
> Reviewed-by: Tommi <tommi@webrtc.org>
> Commit-Queue: Fredrik Solenberg <solenberg@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#20690}

TBR=solenberg@webrtc.org,kwiberg@webrtc.org,tommi@webrtc.org

Change-Id: I2c8caea55f3c6f778b0db064cc43349abbe96fb0
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:8529
Reviewed-on: https://webrtc-review.googlesource.com/23460
Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
Commit-Queue: Fredrik Solenberg <solenberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20691}
This commit is contained in:
Fredrik Solenberg 2017-11-15 20:16:45 +00:00 committed by Commit Bot
parent b18e868aa4
commit 6dfb5a8571
2 changed files with 6 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
rtc_base/logging.h
View File

@ -61,12 +61,6 @@
#include "rtc_base/constructormagic.h"
#include "rtc_base/thread_annotations.h"
#if !defined(NDEBUG) || defined(DLOG_ALWAYS_ON)
#define RTC_DLOG_IS_ON 1
#else
#define RTC_DLOG_IS_ON 0
#endif
namespace rtc {
///////////////////////////////////////////////////////////////////////////////
@ -366,24 +360,7 @@ inline bool LogCheckLevel(LoggingSeverity sev) {
#define RTC_PLOG(sev, err) \
RTC_LOG_ERR_EX(sev, err)
// The RTC_DLOG macros are equivalent to their RTC_LOG counterparts except that
// they only generate code in debug builds.
#if RTC_DLOG_IS_ON
#define RTC_DLOG(sev) RTC_LOG(sev)
#define RTC_DLOG_V(sev) RTC_LOG_V(sev)
#define RTC_DLOG_F(sev) RTC_LOG_F(sev)
#define RTC_DLOG_T_F(sev) RTC_LOG_T_F(sev)
#else
#define RTC_DLOG_EAT_STREAM_PARAMS(sev) \
(true ? true : ((void)(rtc::sev), true)) \
? static_cast<void>(0) \
: rtc::LogMessageVoidify() & \
rtc::LogMessage(__FILE__, __LINE__, rtc::sev).stream()
#define RTC_DLOG(sev) RTC_DLOG_EAT_STREAM_PARAMS(sev)
#define RTC_DLOG_V(sev) RTC_DLOG_EAT_STREAM_PARAMS(sev)
#define RTC_DLOG_F(sev) RTC_DLOG_EAT_STREAM_PARAMS(sev)
#define RTC_DLOG_T_F(sev) RTC_DLOG_EAT_STREAM_PARAMS(sev)
#endif
// TODO(?): Add an "assert" wrapper that logs in the same manner.
} // namespace rtc

10
voice_engine/transmit_mixer.cc
View File

@ -32,8 +32,8 @@ TransmitMixer::Create(TransmitMixer*& mixer)
mixer = new TransmitMixer();
if (mixer == NULL)
{
RTC_DLOG(LS_ERROR) <<
"TransmitMixer::Create() unable to allocate memory for mixer";
RTC_LOG(LS_ERROR) << "TransmitMixer::Create() unable to allocate memory "
"for mixer";
return -1;
}
return 0;
@ -187,8 +187,8 @@ void TransmitMixer::ProcessAudio(int delay_ms, int clock_drift,
GainControl* agc = audioproc_->gain_control();
if (agc->set_stream_analog_level(current_mic_level) != 0) {
RTC_DLOG(LS_ERROR) << "set_stream_analog_level failed: current_mic_level = "
<< current_mic_level;
RTC_LOG(LS_ERROR) << "set_stream_analog_level failed: current_mic_level = "
<< current_mic_level;
assert(false);
}
@ -201,7 +201,7 @@ void TransmitMixer::ProcessAudio(int delay_ms, int clock_drift,
int err = audioproc_->ProcessStream(&_audioFrame);
if (err != 0) {
RTC_DLOG(LS_ERROR) << "ProcessStream() error: " << err;
RTC_LOG(LS_ERROR) << "ProcessStream() error: " << err;
assert(false);
}