[trafficserver] branch master updated: Remove Http2ConnectionState::continued_buffer

[ma...@apache.org]

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

maskit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new e4a94b9  Remove Http2ConnectionState::continued_buffer
e4a94b9 is described below

commit e4a94b9cec7119c96fde1205611cb60ceed8428d
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
---
 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 a832216..bb2a2b2 100644
--- a/proxy/http2/Http2ConnectionState.h
+++ b/proxy/http2/Http2ConnectionState.h
@@ -130,11 +130,7 @@ public:
   {
     local_hpack_handle  = new HpackHandle(HTTP2_HEADER_TABLE_SIZE);
     remote_hpack_handle = new HpackHandle(HTTP2_HEADER_TABLE_SIZE);
-
-    continued_buffer.iov_base = nullptr;
-    continued_buffer.iov_len  = 0;
-
-    dependency_tree = new DependencyTree(Http2::max_concurrent_streams_in);
+    dependency_tree     = new DependencyTree(Http2::max_concurrent_streams_in);
   }
 
   void
@@ -148,9 +144,6 @@ public:
     mutex = nullptr; // magic happens - assigning to nullptr frees the ProxyMutex
     delete local_hpack_handle;
     delete remote_hpack_handle;
-
-    ats_free(continued_buffer.iov_base);
-
     delete dependency_tree;
     this->ua_session = nullptr;
 
@@ -326,7 +319,6 @@ private:
   //     "If the END_HEADERS bit is not set, this frame MUST be followed by
   //     another CONTINUATION frame."
   Http2StreamId continued_stream_id = 0;
-  IOVec continued_buffer;
   bool _scheduled                   = false;
   bool fini_received                = false;
   int recursion                     = 0;