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

diff --git a/folly/experimental/logging/LogStreamProcessor.cpp b/folly/experimental/logging/LogStreamProcessor.cpp
index e70df787fa8e3af2522196b3e4c03f3e5abd949e..84c8e0d6882e49ee23cf890784f889690fcd724a 100644 (file)
--- a/folly/experimental/logging/LogStreamProcessor.cpp
+++ b/folly/experimental/logging/LogStreamProcessor.cpp
@@ -42,7 +42,7 @@ void LogStreamProcessor::operator&(LogStream&& stream) noexcept {
   // no streaming arguments were supplied to the logging macro.
   // Therefore we don't bother calling extractMessageString(stream),
   // and just directly use message_.
-  assert(stream.empty());
+  DCHECK(stream.empty());
 
   category_->processMessage(LogMessage{
       category_, level_, filename_, lineNumber_, std::move(message_)});