Commit 5086391b authored by Jonathan Kron's avatar Jonathan Kron Committed by Facebook Github Bot

Fixing opt-asan/ubsan fail for folly contbuild

Summary: Flag caused compilation error on unused param, asserts optimized out in opt-asan/ubsan builds.  Replaced with DCHECK as per meyering's advice.

Reviewed By: meyering

Differential Revision: D5246089

fbshipit-source-id: dc0abda91f900dd98af31410f4667c52404997f5
parent 626f5abb
...@@ -42,7 +42,7 @@ void LogStreamProcessor::operator&(LogStream&& stream) noexcept { ...@@ -42,7 +42,7 @@ void LogStreamProcessor::operator&(LogStream&& stream) noexcept {
// no streaming arguments were supplied to the logging macro. // no streaming arguments were supplied to the logging macro.
// Therefore we don't bother calling extractMessageString(stream), // Therefore we don't bother calling extractMessageString(stream),
// and just directly use message_. // and just directly use message_.
assert(stream.empty()); DCHECK(stream.empty());
category_->processMessage(LogMessage{ category_->processMessage(LogMessage{
category_, level_, filename_, lineNumber_, std::move(message_)}); category_, level_, filename_, lineNumber_, std::move(message_)});
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment