You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2018/08/03 21:51:09 UTC

[trafficserver] branch 7.1.x updated: Remove Http2ConnectionState::continued_buffer

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

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/7.1.x by this push:
     new 63b7c80  Remove Http2ConnectionState::continued_buffer
63b7c80 is described below

commit 63b7c80d6127708f0d4a04b92a111fa975891c0d
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Sun Jul 15 23:10:58 2018 +0900

    Remove Http2ConnectionState::continued_buffer
    
    continued_buffer is not used at all
    
    (cherry picked from commit e4a94b9cec7119c96fde1205611cb60ceed8428d)
    
    Conflicts:
    	proxy/http2/Http2ConnectionState.h
---
 proxy/http2/Http2ConnectionState.h | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/proxy/http2/Http2ConnectionState.h b/proxy/http2/Http2ConnectionState.h
index 68070ad..2410c61 100644
--- a/proxy/http2/Http2ConnectionState.h
+++ b/proxy/http2/Http2ConnectionState.h
@@ -146,11 +146,7 @@ public:
   {
     local_hpack_handle  = new HpackHandle(HTTP2_HEADER_TABLE_SIZE);
     remote_hpack_handle = new HpackHandle(HTTP2_HEADER_TABLE_SIZE);
-
-    continued_buffer.iov_base = NULL;
-    continued_buffer.iov_len  = 0;
-
-    dependency_tree = new DependencyTree(Http2::max_concurrent_streams_in);
+    dependency_tree     = new DependencyTree(Http2::max_concurrent_streams_in);
   }
 
   void
@@ -161,9 +157,6 @@ public:
     mutex = NULL; // magic happens - assigning to NULL frees the ProxyMutex
     delete local_hpack_handle;
     delete remote_hpack_handle;
-
-    ats_free(continued_buffer.iov_base);
-
     delete dependency_tree;
     this->ua_session = nullptr;
 
@@ -305,7 +298,6 @@ private:
   //     "If the END_HEADERS bit is not set, this frame MUST be followed by
   //     another CONTINUATION frame."
   Http2StreamId continued_stream_id;
-  IOVec continued_buffer;
   bool _scheduled;
   bool fini_received;
   int recursion;