Index: webrtc/modules/rtp_rtcp/source/producer_fec.cc |
diff --git a/webrtc/modules/rtp_rtcp/source/producer_fec.cc b/webrtc/modules/rtp_rtcp/source/producer_fec.cc |
index c7ea19db5862f4ce6b97184548f900bef54aa154..b259da60d960af83cab9d480e510a883a1bf6300 100644 |
--- a/webrtc/modules/rtp_rtcp/source/producer_fec.cc |
+++ b/webrtc/modules/rtp_rtcp/source/producer_fec.cc |
@@ -139,10 +139,11 @@ int ProducerFec::AddRtpPacketAndGenerateFec(const uint8_t* data_buffer, |
const bool marker_bit = (data_buffer[1] & kRtpMarkerBitMask) ? true : false; |
if (media_packets_fec_.size() < ForwardErrorCorrection::kMaxMediaPackets) { |
// Generic FEC can only protect up to kMaxMediaPackets packets. |
- ForwardErrorCorrection::Packet* packet = new ForwardErrorCorrection::Packet; |
+ std::unique_ptr<ForwardErrorCorrection::Packet> packet( |
+ new ForwardErrorCorrection::Packet); |
stefan-webrtc
2016/06/29 19:36:18
()
brandtr
2016/06/30 14:05:55
Done.
|
packet->length = payload_length + rtp_header_length; |
memcpy(packet->data, data_buffer, packet->length); |
- media_packets_fec_.push_back(packet); |
+ media_packets_fec_.push_back(std::move(packet)); |
} |
if (marker_bit) { |
++num_frames_; |
@@ -218,7 +219,7 @@ std::vector<RedPacket*> ProducerFec::GetFecPackets(int red_pl_type, |
// media packet. |
ForwardErrorCorrection::Packet* packet_to_send = fec_packets_.front(); |
ForwardErrorCorrection::Packet* last_media_packet = |
- media_packets_fec_.back(); |
+ media_packets_fec_.back().get(); |
RedPacket* red_packet = new RedPacket( |
packet_to_send->length + kREDForFECHeaderLength + rtp_header_length); |
@@ -250,10 +251,7 @@ int ProducerFec::Overhead() const { |
} |
void ProducerFec::DeletePackets() { |
- while (!media_packets_fec_.empty()) { |
- delete media_packets_fec_.front(); |
- media_packets_fec_.pop_front(); |
- } |
+ media_packets_fec_.clear(); |
assert(media_packets_fec_.empty()); |
} |