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

Unified Diff: net/quic/core/reliable_quic_stream.cc

Issue 2314863003: Fix all Performance and Bug-Prone patterns in quic/core. No functional change. Not flag protected. (Closed)
Patch Set: Created 4 years, 3 months 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
« no previous file with comments | « net/quic/core/quic_sent_packet_manager.cc ('k') | net/tools/quic/chlo_extractor.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/reliable_quic_stream.cc
diff --git a/net/quic/core/reliable_quic_stream.cc b/net/quic/core/reliable_quic_stream.cc
index 3f1ae0a36981f98badbfa3fd84d83ad484b6222b..ed110e2c2645e1f31ac20a7e9383b2fcaef1ff64 100644
--- a/net/quic/core/reliable_quic_stream.cc
+++ b/net/quic/core/reliable_quic_stream.cc
@@ -46,7 +46,9 @@ size_t GetReceivedFlowControlWindow(QuicSession* session) {
ReliableQuicStream::PendingData::PendingData(
string data_in,
QuicAckListenerInterface* ack_listener_in)
- : data(data_in), offset(0), ack_listener(ack_listener_in) {}
+ : data(std::move(data_in)),
+ offset(0),
+ ack_listener(ack_listener_in) {}
ReliableQuicStream::PendingData::~PendingData() {}
« no previous file with comments | « net/quic/core/quic_sent_packet_manager.cc ('k') | net/tools/quic/chlo_extractor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698