http2: eliminate dead code

PR-URL: #15693 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
nodejs · Oct 11, 2017 · 68cd233 · 68cd233
1 parent 078ee27
commit 68cd233
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 20 deletions.
diff --git a/src/node_http2.cc b/src/node_http2.cc
@@ -25,9 +25,6 @@ Freelist<Nghttp2Stream, FREELIST_MAX> stream_free_list;
 
 Freelist<nghttp2_header_list, FREELIST_MAX> header_free_list;
 
-Freelist<nghttp2_data_chunks_t, FREELIST_MAX>
-    data_chunks_free_list;
-
 Nghttp2Session::Callbacks Nghttp2Session::callback_struct_saved[2] = {
     Callbacks(false),
     Callbacks(true)};

diff --git a/src/node_http2_core-inl.h b/src/node_http2_core-inl.h
@@ -30,9 +30,6 @@ extern Freelist<Nghttp2Stream, FREELIST_MAX> stream_free_list;
 
 extern Freelist<nghttp2_header_list, FREELIST_MAX> header_free_list;
 
-extern Freelist<nghttp2_data_chunks_t, FREELIST_MAX>
-    data_chunks_free_list;
-
 #ifdef NODE_DEBUG_HTTP2
 inline int Nghttp2Session::OnNghttpError(nghttp2_session* session,
                                          const char* message,
@@ -905,12 +902,6 @@ inline void Nghttp2Stream::ReadStop() {
                                         id_, 0);
 }
 
-nghttp2_data_chunks_t::~nghttp2_data_chunks_t() {
-  for (unsigned int n = 0; n < nbufs; n++) {
-    free(buf[n].base);
-  }
-}
-
 Nghttp2Session::Callbacks::Callbacks(bool kHasGetPaddingCallback) {
   nghttp2_session_callbacks_new(&callbacks);
   nghttp2_session_callbacks_set_on_begin_headers_callback(

diff --git a/src/node_http2_core.h b/src/node_http2_core.h
@@ -40,7 +40,6 @@ class Nghttp2Stream;
 
 struct nghttp2_stream_write_t;
 struct nghttp2_data_chunk_t;
-struct nghttp2_data_chunks_t;
 
 #define MAX_BUFFER_COUNT 10
 #define SEND_BUFFER_RECOMMENDED_SIZE 4096
@@ -508,13 +507,6 @@ struct nghttp2_data_chunk_t {
   nghttp2_data_chunk_t* next = nullptr;
 };
 
-struct nghttp2_data_chunks_t {
-  unsigned int nbufs = 0;
-  uv_buf_t buf[MAX_BUFFER_COUNT];
-
-  inline ~nghttp2_data_chunks_t();
-};
-
 }  // namespace http2
 }  // namespace node