Skip to content

Commit f955d12

Browse files
generatedunixname89002005287564facebook-github-bot
authored andcommitted
Fix CQS signal modernize-use-emplace in fbcode/quic/state
Differential Revision: D82288893 fbshipit-source-id: d0d01560b3b06852ee2aa2dd70ce40872a941ac7
1 parent 0c2e459 commit f955d12

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

quic/state/QuicAckFrequencyFunctions.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ void requestPeerAckFrequencyChange(
2525
frame.updateMaxAckDelay = maxAckDelay.count();
2626
frame.reorderThreshold = reorderThreshold;
2727
frame.sequenceNumber = conn.nextAckFrequencyFrameSequenceNumber++;
28-
conn.pendingEvents.frames.push_back(frame);
28+
conn.pendingEvents.frames.emplace_back(frame);
2929
}
3030

3131
std::chrono::microseconds clampMaxAckDelay(

quic/state/SimpleFrameFunctions.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ void updateSimpleFrameOnPacketLoss(
7878
case QuicSimpleFrame::Type::StopSendingFrame: {
7979
const StopSendingFrame& stopSendingFrame = *frame.asStopSendingFrame();
8080
if (conn.streamManager->streamExists(stopSendingFrame.streamId)) {
81-
conn.pendingEvents.frames.push_back(stopSendingFrame);
81+
conn.pendingEvents.frames.emplace_back(stopSendingFrame);
8282
}
8383
break;
8484
}
@@ -91,11 +91,11 @@ void updateSimpleFrameOnPacketLoss(
9191
break;
9292
}
9393
case QuicSimpleFrame::Type::PathResponseFrame: {
94-
conn.pendingEvents.frames.push_back(*frame.asPathResponseFrame());
94+
conn.pendingEvents.frames.emplace_back(*frame.asPathResponseFrame());
9595
break;
9696
}
9797
case QuicSimpleFrame::Type::HandshakeDoneFrame: {
98-
conn.pendingEvents.frames.push_back(*frame.asHandshakeDoneFrame());
98+
conn.pendingEvents.frames.emplace_back(*frame.asHandshakeDoneFrame());
9999
break;
100100
}
101101
case QuicSimpleFrame::Type::NewConnectionIdFrame:

quic/state/StateData.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ QuicConnectionStateBase::retireAndSwitchPeerConnectionIds() {
142142
return false;
143143
}
144144
DCHECK(currentConnIdDataIt != end);
145-
pendingEvents.frames.push_back(
145+
pendingEvents.frames.emplace_back(
146146
RetireConnectionIdFrame(currentConnIdDataIt->sequenceNumber));
147147
mainPeerId = replacementConnIdDataIt->connId;
148148

0 commit comments

Comments
 (0)