modules/rtp_rtcp/include folder cleared of lint warnings

webrtc/modules/rtp_rtcp/mocks/mock_rtp_rtcp.h

Index: webrtc/modules/rtp_rtcp/mocks/mock_rtp_rtcp.h
diff --git a/webrtc/modules/rtp_rtcp/mocks/mock_rtp_rtcp.h b/webrtc/modules/rtp_rtcp/mocks/mock_rtp_rtcp.h
index ca5434ee6d05a6a80153695ecf57730ce8a70772..754f0fa57d1d3a211f96640b0851b184da6be6b5 100644
--- a/webrtc/modules/rtp_rtcp/mocks/mock_rtp_rtcp.h
+++ b/webrtc/modules/rtp_rtcp/mocks/mock_rtp_rtcp.h
@@ -214,8 +214,7 @@ class MockRtpRtcp : public RtpRtcp {
       int32_t(const uint8_t key, const uint16_t time_ms, const uint8_t level));
   MOCK_METHOD1(SetSendREDPayloadType,
       int32_t(const int8_t payloadType));
-  MOCK_CONST_METHOD1(SendREDPayloadType,
-      int32_t(int8_t& payloadType));
+  MOCK_CONST_METHOD1(SendREDPayloadType, int32_t(int8_t* payloadType));
   MOCK_METHOD2(SetRTPAudioLevelIndicationStatus,
       int32_t(const bool enable, const uint8_t ID));
   MOCK_CONST_METHOD2(GetRTPAudioLevelIndicationStatus,
@@ -229,9 +228,9 @@ class MockRtpRtcp : public RtpRtcp {
                     const uint8_t payload_type_red,
                     const uint8_t payload_type_fec));
   MOCK_METHOD3(GenericFECStatus,

[mflodman, 2015/12/09 09:19:13]

I think you should keep the old method too until it's removed from the
interface.

[danilchap, 2015/12/09 11:23:36]

GMock documentation tells that virtual functions should be mocked.
Old method became a non-virtual inline that will automatically call new method.

[mflodman, 2015/12/10 08:24:23]

Acknowledged.

-               void(bool& enable,
-                    uint8_t& payloadTypeRED,
-                    uint8_t& payloadTypeFEC));
+               void(bool* enable,
+                    uint8_t* payloadTypeRED,
+                    uint8_t* payloadTypeFEC));
   MOCK_METHOD2(SetFecParameters,
       int32_t(const FecProtectionParams* delta_params,
               const FecProtectionParams* key_params));