From 2bc91e8c6af8f9a34699e90f11b6ff03ab410b6f Mon Sep 17 00:00:00 2001 From: Evan Shrubsole Date: Fri, 17 Jan 2020 11:44:24 +0100 Subject: [PATCH] Avoid extra EncodedFrame copy in RunPostEncode MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit All uses of encoded_image are const, except for the copy for running on the encoder_queue_. Bug: None Change-Id: I7fc8cb46f6afb42a2d27961d3d3ff8d9e63fe1b5 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/166442 Reviewed-by: Erik Språng Commit-Queue: Evan Shrubsole Cr-Commit-Position: refs/heads/master@{#30351} --- video/video_stream_encoder.cc | 2 +- video/video_stream_encoder.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/video/video_stream_encoder.cc b/video/video_stream_encoder.cc index 486143c5d8..68f24e6944 100644 --- a/video/video_stream_encoder.cc +++ b/video/video_stream_encoder.cc @@ -1761,7 +1761,7 @@ void VideoStreamEncoder::OnVideoSourceRestrictionsUpdated( video_source_sink_controller_->PushSourceSinkSettings(); } -void VideoStreamEncoder::RunPostEncode(EncodedImage encoded_image, +void VideoStreamEncoder::RunPostEncode(const EncodedImage& encoded_image, int64_t time_sent_us, int temporal_index, DataSize frame_size) { diff --git a/video/video_stream_encoder.h b/video/video_stream_encoder.h index 5ac6db8028..a90542087f 100644 --- a/video/video_stream_encoder.h +++ b/video/video_stream_encoder.h @@ -195,7 +195,7 @@ class VideoStreamEncoder : public VideoStreamEncoderInterface, void SetEncoderRates(const EncoderRateSettings& rate_settings) RTC_RUN_ON(&encoder_queue_); - void RunPostEncode(EncodedImage encoded_image, + void RunPostEncode(const EncodedImage& encoded_image, int64_t time_sent_us, int temporal_index, DataSize frame_size);