Reverse |rtx_payload_types| map, and rename.

webrtc/call/call.cc

Index: webrtc/call/call.cc
diff --git a/webrtc/call/call.cc b/webrtc/call/call.cc
index 1b5bc456b0f1a722436902db740881233f5dbee1..a315d744d6080da0e3f1065ffca1c643fa0ee65f 100644
--- a/webrtc/call/call.cc
+++ b/webrtc/call/call.cc
@@ -87,6 +87,15 @@ bool UseSendSideBwe(const FlexfecReceiveStream::Config& config) {
   return UseSendSideBwe(config.rtp_header_extensions, config.transport_cc);
 }
 
+template <typename K, typename V>

[danilchap, 2017/08/21 14:11:26]

may be do not use template until you'll use it for different K and V

[nisse-webrtc, 2017/08/22 07:11:01]

Done.

+const K* find_by_value(const std::map<K, V>& m, V v) {

[danilchap, 2017/08/21 14:11:26]

FindByValue
https://google.github.io/styleguide/cppguide.html#Function_Names
Or may be FindKeyByValue (since result is pointer to key, not iterator)

[nisse-webrtc, 2017/08/22 07:11:01]

Done.

+  for (const auto& kv : m) {
+    if (kv.second == v)
+      return &kv.first;
+  }
+  return nullptr;
+}
+
 rtclog::StreamConfig CreateRtcLogStreamConfig(
     const VideoReceiveStream::Config& config) {
   rtclog::StreamConfig rtclog_config;
@@ -98,10 +107,9 @@ rtclog::StreamConfig CreateRtcLogStreamConfig(
   rtclog_config.rtp_extensions = config.rtp.extensions;
 
   for (const auto& d : config.decoders) {
-    auto search = config.rtp.rtx_payload_types.find(d.payload_type);
-    rtclog_config.codecs.emplace_back(
-        d.payload_name, d.payload_type,
-        search != config.rtp.rtx_payload_types.end() ? search->second : 0);
+    auto search = find_by_value(config.rtp.media_pt_by_rtx_pt, d.payload_type);

[danilchap, 2017/08/21 14:11:26]

since search is no longer a long-named iterator, use exact type: 'const int*'
(or at least auto*)

[nisse-webrtc, 2017/08/22 07:11:01]

Done.

+    rtclog_config.codecs.emplace_back(d.payload_name, d.payload_type,
+                                      search ? *search : 0);
   }
   return rtclog_config;
 }