| commit | 05cf6be7263ce11f4bd5222c9a3a4cc2a1470a01 | [log] [tgz] |
|---|---|---|
| author | Mirko Bonadei <mbonadei@webrtc.org> | Thu Jan 31 20:38:12 2019 |
| committer | Commit Bot <commit-bot@chromium.org> | Fri Feb 01 15:02:36 2019 |
| tree | 97bdadbc0ec80c13f2afff73b349e68b397fa19b | |
| parent | 9e24fc21960cf9dc6f9171f2951d538b95e637c9 [diff] [blame] |
[clang-tidy] Apply performance-move-const-arg fixes. 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 are some wrong fixes to correct, this CL collects all the fixes that could be landed as is. [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: Ic4882213556344e65c66e27415e91ff6f89134d7 Reviewed-on: https://webrtc-review.googlesource.com/c/120814 Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org> Cr-Commit-Position: refs/heads/master@{#26515}
diff --git a/call/simulated_network.cc b/call/simulated_network.cc index 9bb8bab..0884b29 100644 --- a/call/simulated_network.cc +++ b/call/simulated_network.cc
@@ -125,7 +125,7 @@ } // Time to get this packet. - PacketInfo packet = std::move(capacity_link_.front()); + PacketInfo packet = capacity_link_.front(); capacity_link_.pop(); time_us += time_until_front_exits_us; @@ -165,7 +165,7 @@ needs_sort = true; } } - delay_link_.emplace_back(std::move(packet)); + delay_link_.emplace_back(packet); } last_capacity_link_visit_us_ = time_now_us; // Cannot save unused capacity for later.