[trafficserver] branch quic-latest updated: Make _has_new_data flag true only if received packet is not ack only

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/quic-latest by this push:
     new da5385f  Make _has_new_data flag true only if received packet is not ack only
da5385f is described below

commit da5385f76a6f1de6e2ecab8792889f33757cbf5b
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Fri Feb 15 11:00:53 2019 +0900

    Make _has_new_data flag true only if received packet is not ack only
---
 iocore/net/quic/QUICAckFrameCreator.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/iocore/net/quic/QUICAckFrameCreator.cc b/iocore/net/quic/QUICAckFrameCreator.cc
index 6f634c5..7214fdc 100644
--- a/iocore/net/quic/QUICAckFrameCreator.cc
+++ b/iocore/net/quic/QUICAckFrameCreator.cc
@@ -198,13 +198,13 @@ QUICAckFrameManager::QUICAckFrameCreator::push_back(QUICPacketNumber packet_numb
   }
 
   if (!ack_only) {
-    this->_available = true;
+    this->_available    = true;
+    this->_has_new_data = true;
   } else {
     this->_should_send = this->_available ? this->_should_send : false;
   }
 
-  this->_has_new_data = true;
-  this->_expect_next  = packet_number + 1;
+  this->_expect_next = packet_number + 1;
   this->_packet_numbers.push_back({ack_only, packet_number});
 }