From 84a7ae21be5dbf363b4a0d7b41262fb4ab136293 Mon Sep 17 00:00:00 2001
From: Daniel Bevenius <daniel.bevenius@gmail.com>
Date: Wed, 14 Nov 2018 05:09:02 +0100
Subject: [PATCH] src: remove unused variables
Currently the following compiler warnings are generated:
../src/node_process.cc:799:16:
warning: unused variable 'ary' [-Wunused-variable]
Local<Array> ary = Array::New(args.GetIsolate());
^
1 warning generated.
../src/node_http2.cc:1294:16:
warning: unused variable 'holder' [-Wunused-variable]
Local<Array> holder = Array::New(isolate);
^
1 warning generated.
This commit removes these unused variables.
---
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 20b634f90fc00d..552903059abce5 100644
--- a/src/node_http2.cc
+++ b/src/node_http2.cc
@@ -1291,7 +1291,6 @@ void Http2Session::HandleHeadersFrame(const nghttp2_frame* frame) {
Local<String> name_str;
Local<String> value_str;
- Local<Array> 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 161833ebbd3843..017d8d0073ac83 100644
--- a/src/node_process.cc
+++ b/src/node_process.cc
@@ -796,7 +796,6 @@ void GetParentProcessId(Local<Name> property,
void GetActiveRequests(const FunctionCallbackInfo<Value>& args) {
Environment* env = Environment::GetCurrent(args);
- Local<Array> ary = Array::New(args.GetIsolate());
std::vector<Local<Value>> request_v;
for (auto w : *env->req_wrap_queue()) {
if (w->persistent().IsEmpty())