From 39a5e7dbb0471313f4f00ee4f1eb51ee0ada61e6 Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Wed, 14 Nov 2018 05:09:02 +0100 Subject: [PATCH] src: remove unused variables MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently the following compiler warnings are generated: ../src/node_process.cc:799:16: warning: unused variable 'ary' [-Wunused-variable] Local ary = Array::New(args.GetIsolate()); ^ 1 warning generated. ../src/node_http2.cc:1294:16: warning: unused variable 'holder' [-Wunused-variable] Local holder = Array::New(isolate); ^ 1 warning generated. This commit removes these unused variables. PR-URL: https://github.com/nodejs/node/pull/24355 Reviewed-By: Colin Ihrig Reviewed-By: Michaƫl Zasso Reviewed-By: Michael Dawson Reviewed-By: Luigi Pinca Reviewed-By: Ruben Bridgewater --- src/node_http2.cc | 1 - src/node_process.cc | 1 - 2 files changed, 2 deletions(-) diff --git a/src/node_http2.cc b/src/node_http2.cc index 20b634f90fc..552903059ab 100644 --- a/src/node_http2.cc +++ b/src/node_http2.cc @@ -1291,7 +1291,6 @@ void Http2Session::HandleHeadersFrame(const nghttp2_frame* frame) { Local name_str; Local value_str; - Local holder = Array::New(isolate); // The headers are passed in above as a queue of nghttp2_header structs. // The following converts that into a JS array with the structure: // [name1, value1, name2, value2, name3, value3, name3, value4] and so on. diff --git a/src/node_process.cc b/src/node_process.cc index 161833ebbd3..017d8d0073a 100644 --- a/src/node_process.cc +++ b/src/node_process.cc @@ -796,7 +796,6 @@ void GetParentProcessId(Local property, void GetActiveRequests(const FunctionCallbackInfo& args) { Environment* env = Environment::GetCurrent(args); - Local ary = Array::New(args.GetIsolate()); std::vector> request_v; for (auto w : *env->req_wrap_queue()) { if (w->persistent().IsEmpty())