Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1153)

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc

Issue 2455963003: Simplify SetFecParameters signature. (Closed)
Patch Set: Fix fuzzer. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc
index 30b4acc9d6f79c58812fde56d056c041bf64b55a..dc568bf3e6f5eeccb27d0d6912f3f9d98eedc7bf 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc
@@ -227,14 +227,12 @@ size_t RTPSenderVideo::FecPacketOverhead() const {
return overhead;
}
-void RTPSenderVideo::SetFecParameters(const FecProtectionParams* delta_params,
- const FecProtectionParams* key_params) {
+void RTPSenderVideo::SetFecParameters(const FecProtectionParams& delta_params,
+ const FecProtectionParams& key_params) {
rtc::CritScope cs(&crit_);
- RTC_DCHECK(delta_params);
- RTC_DCHECK(key_params);
if (ulpfec_enabled()) {
- delta_fec_params_ = *delta_params;
- key_fec_params_ = *key_params;
+ delta_fec_params_ = delta_params;
+ key_fec_params_ = key_params;
}
}
@@ -292,8 +290,8 @@ bool RTPSenderVideo::SendVideo(RtpVideoCodecTypes video_type,
bool first_frame = first_frame_sent_();
{
rtc::CritScope cs(&crit_);
- FecProtectionParams* fec_params =
- frame_type == kVideoFrameKey ? &key_fec_params_ : &delta_fec_params_;
+ const FecProtectionParams& fec_params =
+ frame_type == kVideoFrameKey ? key_fec_params_ : delta_fec_params_;
ulpfec_generator_.SetFecParameters(fec_params);
storage = packetizer->GetStorageType(retransmission_settings_);
red_enabled = this->red_enabled();
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_sender_video.h ('k') | webrtc/modules/rtp_rtcp/source/ulpfec_generator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698