From 411695e1d26565c7498992be19731bbea6b6aa58 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Thu, 12 Oct 2017 12:04:02 +0200 Subject: [PATCH] build: lint benchmark addon PR-URL: https://github.com/nodejs/node/pull/16160 Reviewed-By: Joyee Cheung Reviewed-By: Franziska Hinkelmann --- Makefile | 1 + benchmark/misc/function_call/binding.cc | 9 +++------ 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/Makefile b/Makefile index 755542eb07..ddbfd0f885 100644 --- a/Makefile +++ b/Makefile @@ -965,6 +965,7 @@ LINT_CPP_EXCLUDE += $(wildcard test/addons-napi/??_*/*.cc test/addons-napi/??_*/ LINT_CPP_EXCLUDE += src/tracing/trace_event.h src/tracing/trace_event_common.h LINT_CPP_FILES = $(filter-out $(LINT_CPP_EXCLUDE), $(wildcard \ + benchmark/misc/function_call/binding.cc \ src/*.c \ src/*.cc \ src/*.h \ diff --git a/benchmark/misc/function_call/binding.cc b/benchmark/misc/function_call/binding.cc index 95d2a56de2..289a94ac3e 100644 --- a/benchmark/misc/function_call/binding.cc +++ b/benchmark/misc/function_call/binding.cc @@ -1,17 +1,14 @@ #include #include -using namespace v8; - static int c = 0; -void Hello(const FunctionCallbackInfo& args) { +void Hello(const v8::FunctionCallbackInfo& args) { args.GetReturnValue().Set(c++); } -extern "C" void init (Local target) { - HandleScope scope(Isolate::GetCurrent()); +void Initialize(v8::Local target) { NODE_SET_METHOD(target, "hello", Hello); } -NODE_MODULE(NODE_GYP_MODULE_NAME, init) +NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)