lib/mvfst
1
0
Fork 0
mirror of https://github.com/facebookincubator/mvfst.git synced 2025-07-29 03:41:11 +03:00
Code Activity

Minor clean up

Browse Source 
Summary: Minor clean up

Reviewed By: mjoras

Differential Revision: D35510673

fbshipit-source-id: fcd6982dd49a08ac2c9e2148b1cfd5beec9f49dd
This commit is contained in:
Konstantin Tsoy
2022-06-15 19:03:04 -07:00
committed by Facebook GitHub Bot
parent 6144ea1ded
commit be0cd3dacc
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
quic/api/QuicPacketScheduler.cpp
View File

@ -245,7 +245,7 @@ SchedulingResult FrameScheduler::scheduleFramesForPacket(
rstWritten = rstScheduler_->writeRsts(wrapper); rstWritten = rstScheduler_->writeRsts(wrapper);
} }
// Long time ago we decided RST has higher priority than Acks. // Long time ago we decided RST has higher priority than Acks.
if (ackScheduler_ && ackScheduler_->hasPendingAcks()) { if (hasPendingAcks()) {
if (cryptoDataWritten || rstWritten) { if (cryptoDataWritten || rstWritten) {
// If packet has non ack data, it is subject to congestion control. We // If packet has non ack data, it is subject to congestion control. We
// need to use the wrapper/ // need to use the wrapper/
@ -317,8 +317,7 @@ void FrameScheduler::writeNextAcks(PacketBuilderInterface& builder) {
} }
bool FrameScheduler::hasData() const { bool FrameScheduler::hasData() const {
return (ackScheduler_ && ackScheduler_->hasPendingAcks()) || return (hasPendingAcks()) || hasImmediateData();
hasImmediateData();
} }
bool FrameScheduler::hasPendingAcks() const { bool FrameScheduler::hasPendingAcks() const {