Simplify SetFecParameters signature.

webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc

Index: webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc b/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc
index 2a2bf0df4b227a2ff9e02083ef4ee48324108d79..a6c62bfd01e63775b71c3e59f941a0d6f3bde563 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc
@@ -61,6 +61,15 @@ RtpRtcp* RtpRtcp::CreateRtpRtcp(const RtpRtcp::Configuration& configuration) {
   }
 }
 
+// Deprecated.
+inline int32_t RtpRtcp::SetFecParameters(

[danilchap, 2016/10/28 11:16:11]

since it is in .cc remove inline (you might get linking problems if you inline
it in .cc but use it in a different compilation unit)

[brandtr, 2016/10/28 12:10:03]

Done.

+    const FecProtectionParams* delta_params,
+    const FecProtectionParams* key_params) {
+  RTC_DCHECK(delta_params);
+  RTC_DCHECK(key_params);
+  return SetFecParameters(*delta_params, *key_params) ? 0 : -1;
+}
+
 ModuleRtpRtcpImpl::ModuleRtpRtcpImpl(const Configuration& configuration)
     : rtp_sender_(configuration.audio,
                   configuration.clock,
@@ -794,9 +803,9 @@ void ModuleRtpRtcpImpl::SetUlpfecConfig(int red_payload_type,
   rtp_sender_.SetUlpfecConfig(red_payload_type, ulpfec_payload_type);
 }
 
-int32_t ModuleRtpRtcpImpl::SetFecParameters(
-    const FecProtectionParams* delta_params,
-    const FecProtectionParams* key_params) {
+bool ModuleRtpRtcpImpl::SetFecParameters(
+    const FecProtectionParams& delta_params,
+    const FecProtectionParams& key_params) {
   return rtp_sender_.SetFecParameters(delta_params, key_params);
 }