| commit | 1c54605e77e07fa9b69b763e63e2af6c4c0d0d8e | [log] [tgz] |
|---|---|---|
| author | Mirko Bonadei <mbonadei@webrtc.org> | Mon Feb 04 14:50:38 2019 +0100 |
| committer | Commit Bot <commit-bot@chromium.org> | Tue Feb 05 15:12:20 2019 +0000 |
| tree | 5783f569ca70b09661bd7a577ec3a7d0ae63892b | |
| parent | 87ce874f99eff5bc200b74a816f8e1c9ed7cacdd [diff] [blame] |
[clang-tidy] Apply performance-move-const-arg fixes (misc). This CL is a manual spin-off of [1], which tried to apply clang-tidy's performance-move-const-arg [1] to the WebRTC codebase. Since there were some wrong fixes to correct, this CL lands a few different fixes, like adding a constructor overload to take an rvalue reference or remove 'const' to make std::move effective. [1] - https://webrtc-review.googlesource.com/c/src/+/120350 [2] - https://clang.llvm.org/extra/clang-tidy/checks/performance-move-const-arg.html Bug: webrtc:10252 Change-Id: I42a777247fee2cb788efcd7c2035148330056b7a Reviewed-on: https://webrtc-review.googlesource.com/c/120928 Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org> Cr-Commit-Position: refs/heads/master@{#26553}
diff --git a/audio/channel_send.cc b/audio/channel_send.cc index dfbbb57..e7408f0 100644 --- a/audio/channel_send.cc +++ b/audio/channel_send.cc
@@ -587,7 +587,7 @@ sampling_rate_hz = media_transport_sampling_frequency_; channel_id = media_transport_channel_id_; } - const MediaTransportEncodedAudioFrame frame( + MediaTransportEncodedAudioFrame frame( /*sampling_rate_hz=*/sampling_rate_hz, // TODO(nisse): Timestamp and sample index are the same for all supported