admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add explicit copy constructors to RTPHeader and RTPHeaderExtension

Browse Source 
Explicit copy-constructors are required by chromium. (No copy constructors were used until now, but a different CL requires them.)

BUG=None
TBR=stefan@webrtc.org

Review-Url: https://codereview.webrtc.org/3006343002
Cr-Commit-Position: refs/heads/master@{#19777}
This commit is contained in:
eladalon 2017-09-11 08:48:26 -07:00 committed by Commit Bot
parent 4bb3b9c6cb
commit 98b1b7d59e
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
webrtc/common_types.cc
View File

@ -57,6 +57,12 @@ RTPHeaderExtension::RTPHeaderExtension()
videoContentType(VideoContentType::UNSPECIFIED),
has_video_timing(false) {}
RTPHeaderExtension::RTPHeaderExtension(const RTPHeaderExtension& other) =
default;
RTPHeaderExtension& RTPHeaderExtension::operator=(
const RTPHeaderExtension& other) = default;
RTPHeader::RTPHeader()
: markerBit(false),
payloadType(0),
@ -70,6 +76,10 @@ RTPHeader::RTPHeader()
payload_type_frequency(0),
extension() {}
RTPHeader::RTPHeader(const RTPHeader& other) = default;
RTPHeader& RTPHeader::operator=(const RTPHeader& other) = default;
VideoCodec::VideoCodec()
: codecType(kVideoCodecUnknown),
plName(),

4
webrtc/common_types.h
View File

@ -792,6 +792,8 @@ typedef StringRtpHeaderExtension Mid;
struct RTPHeaderExtension {
RTPHeaderExtension();
RTPHeaderExtension(const RTPHeaderExtension& other);
RTPHeaderExtension& operator=(const RTPHeaderExtension& other);
bool hasTransmissionTimeOffset;
int32_t transmissionTimeOffset;
@ -835,6 +837,8 @@ struct RTPHeaderExtension {
struct RTPHeader {
RTPHeader();
RTPHeader(const RTPHeader& other);
RTPHeader& operator=(const RTPHeader& other);
bool markerBit;
uint8_t payloadType;