commit 8aba6b41144360a375e87990f680cde9b56f88d9
author Danil Chapovalov <danilchap@webrtc.org> Tue Apr 17 08:10:57 2018
committer Commit Bot <commit-bot@chromium.org> Tue Apr 17 12:05:13 2018
tree 6811caa38b68276830c58ee6e9e3b367a0b6fa56
parent 70595c1c3523b2848ebbd7e6d72cf2e445261d14

Remove incompatiblities with absl::optional in audio_coding

PCMFile.cc uses RTC_DCHECK. include and depend on rtc_base:checks target directly

change usage of value_or by using explicit constructor instead of implicit

Bug: webrtc:9078
Change-Id: I63c596b8a05b387e56df846b15c33a605fbad4e6
Reviewed-on: https://webrtc-review.googlesource.com/69985
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22897}

modules/audio_coding/BUILD.gn

diff --git a/modules/audio_coding/BUILD.gn b/modules/audio_coding/BUILD.gn
index c350d76..9c87887 100644
--- a/modules/audio_coding/BUILD.gn
+++ b/modules/audio_coding/BUILD.gn
@@ -1382,6 +1382,7 @@
       "../../api:optional",
       "../../api/audio:audio_frame_api",
       "../../api/audio_codecs:builtin_audio_decoder_factory",
+      "../../rtc_base:checks",
       "../../rtc_base:rtc_base_approved",
       "../../rtc_base/synchronization:rw_lock_wrapper",
       "../../system_wrappers",
@@ -1496,6 +1497,7 @@
       "../../api:optional",
       "../../api/audio:audio_frame_api",
       "../../api/audio_codecs:builtin_audio_decoder_factory",
+      "../../rtc_base:checks",
       "../../rtc_base:rtc_base_approved",
       "../../system_wrappers",
       "../../system_wrappers:system_wrappers_default",
@@ -1530,6 +1532,7 @@
       "../../api:optional",
       "../../api/audio:audio_frame_api",
       "../../api/audio_codecs:builtin_audio_decoder_factory",
+      "../../rtc_base:checks",
       "../../rtc_base:rtc_base_approved",
       "../../system_wrappers",
       "../../system_wrappers:system_wrappers_default",

modules/audio_coding/neteq/decoder_database.cc

diff --git a/modules/audio_coding/neteq/decoder_database.cc b/modules/audio_coding/neteq/decoder_database.cc
index 40b0dc7..5b940ae 100644
--- a/modules/audio_coding/neteq/decoder_database.cc
+++ b/modules/audio_coding/neteq/decoder_database.cc
@@ -230,7 +230,8 @@
   }
 
   const auto opt_db_format = NetEqDecoderToSdpAudioFormat(codec_type);
-  const SdpAudioFormat format = opt_db_format.value_or({"arbitrary", 0, 0});
+  const SdpAudioFormat format =
+      opt_db_format.value_or(SdpAudioFormat("arbitrary", 0, 0));
 
   std::pair<DecoderMap::iterator, bool> ret;
   DecoderInfo info(format, decoder, codec_name);

modules/audio_coding/test/PCMFile.cc

diff --git a/modules/audio_coding/test/PCMFile.cc b/modules/audio_coding/test/PCMFile.cc
index bdb46eb..2b2f1f0 100644
--- a/modules/audio_coding/test/PCMFile.cc
+++ b/modules/audio_coding/test/PCMFile.cc
@@ -14,6 +14,7 @@
 #include <stdio.h>
 #include <string.h>
 
+#include "rtc_base/checks.h"
 #include "test/gtest.h"
 
 namespace webrtc {