Add explicit copy constructors to RTPHeader and RTPHeaderExtension

webrtc/common_types.h

Index: webrtc/common_types.h
diff --git a/webrtc/common_types.h b/webrtc/common_types.h
index 49b42ac37656091140a5d2a1476fd9a20fac4b13..0a98b4e8680a6cbf31c3b1ac357a688878492dbc 100644
--- a/webrtc/common_types.h
+++ b/webrtc/common_types.h
@@ -792,6 +792,7 @@ typedef StringRtpHeaderExtension Mid;
 
 struct RTPHeaderExtension {
   RTPHeaderExtension();
+  RTPHeaderExtension(const RTPHeaderExtension& other);

[danilchap, 2017/09/11 13:34:21]

https://google.github.io/styleguide/cppguide.html#Copyable_Movable_Types

"If you define a copy constructor, define the corresponding assignment operator"

[eladalon, 2017/09/11 14:15:03]

Done.

 
   bool hasTransmissionTimeOffset;
   int32_t transmissionTimeOffset;
@@ -835,6 +836,7 @@ struct RTPHeaderExtension {
 
 struct RTPHeader {
   RTPHeader();
+  RTPHeader(const RTPHeader& other);
 
   bool markerBit;
   uint8_t payloadType;