You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ma...@apache.org on 2019/02/15 01:03:59 UTC

[trafficserver] branch quic-latest updated: Cleanup: remove unused counter

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 32aaa25  Cleanup: remove unused counter
32aaa25 is described below

commit 32aaa2520fda1b44dd6b4ec9ce1d59ef4983fa27
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Fri Feb 15 09:56:32 2019 +0900

    Cleanup: remove unused counter
    
    QUICNetVConnection::_handshake_packets_sent is deprecated in favor
    of 2bebf6c7338e28c6ef0f8c1cdb3fb6c4e57fcc26.
---
 iocore/net/P_QUICNetVConnection.h | 4 +---
 iocore/net/QUICNetVConnection.cc  | 5 -----
 2 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/iocore/net/P_QUICNetVConnection.h b/iocore/net/P_QUICNetVConnection.h
index e20da7f..e35c54c 100644
--- a/iocore/net/P_QUICNetVConnection.h
+++ b/iocore/net/P_QUICNetVConnection.h
@@ -375,9 +375,7 @@ private:
   size_t _av_token_len           = 0;
   bool _is_resumption_token_sent = false;
 
-  // This is for limiting number of packets that a server can send without path validation
-  uint32_t _handshake_packets_sent = 0;
-  uint64_t _stream_frames_sent     = 0;
+  uint64_t _stream_frames_sent = 0;
 
   // TODO: Source addresses verification through an address validation token
   bool _has_ack_eliciting_packet_out = true;
diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 28d784c..0003181 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -1252,11 +1252,6 @@ QUICNetVConnection::_state_common_send_packet()
       QUICPacketUPtr packet = this->_packetize_frames(level, max_packet_size);
 
       if (packet) {
-        if (this->netvc_context == NET_VCONNECTION_IN &&
-            (packet->type() == QUICPacketType::INITIAL || packet->type() == QUICPacketType::HANDSHAKE)) {
-          ++this->_handshake_packets_sent;
-        }
-
         if (this->netvc_context == NET_VCONNECTION_IN && !this->_verfied_state.is_verified()) {
           QUICConDebug("send to unverified window: %u", this->_verfied_state.windows());
           this->_verfied_state.consume(packet->size());