From 5d0cee46bb90084e6dcd584deb5bc893862ce3b3 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Tue, 16 Jun 2015 16:11:51 +0200 Subject: [PATCH] vm: remove unnecessary HandleScopes The accessors run inside an implicit HandleScope, there is no need to create a new one. PR-URL: https://github.com/nodejs/io.js/pull/2001 Reviewed-By: Trevor Norris --- src/node_contextify.cc | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/node_contextify.cc b/src/node_contextify.cc index 81d0388a35f390..b52062e72a4614 100644 --- a/src/node_contextify.cc +++ b/src/node_contextify.cc @@ -362,7 +362,6 @@ class ContextifyContext { Local property, const PropertyCallbackInfo& args) { Isolate* isolate = args.GetIsolate(); - HandleScope scope(isolate); ContextifyContext* ctx = Unwrap(args.Data().As()); @@ -387,7 +386,6 @@ class ContextifyContext { Local value, const PropertyCallbackInfo& args) { Isolate* isolate = args.GetIsolate(); - HandleScope scope(isolate); ContextifyContext* ctx = Unwrap(args.Data().As()); @@ -400,7 +398,6 @@ class ContextifyContext { Local property, const PropertyCallbackInfo& args) { Isolate* isolate = args.GetIsolate(); - HandleScope scope(isolate); ContextifyContext* ctx = Unwrap(args.Data().As()); @@ -422,7 +419,6 @@ class ContextifyContext { Local property, const PropertyCallbackInfo& args) { Isolate* isolate = args.GetIsolate(); - HandleScope scope(isolate); ContextifyContext* ctx = Unwrap(args.Data().As()); @@ -435,8 +431,6 @@ class ContextifyContext { static void GlobalPropertyEnumeratorCallback( const PropertyCallbackInfo& args) { - HandleScope scope(args.GetIsolate()); - ContextifyContext* ctx = Unwrap(args.Data().As());