From be0cd3dacc166695d3b97fe9ee4c7142a4ceeccc Mon Sep 17 00:00:00 2001
From: Konstantin Tsoy <ktsoy@fb.com>
Date: Wed, 15 Jun 2022 19:03:04 -0700
Subject: [PATCH] Minor clean up

Summary: Minor clean up

Reviewed By: mjoras

Differential Revision: D35510673

fbshipit-source-id: fcd6982dd49a08ac2c9e2148b1cfd5beec9f49dd
---
 quic/api/QuicPacketScheduler.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

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