optimize HandleScope usage in JSGraph edge processing

Author: mertcanaltin

src/heap_utils.cc

@@ -169,37 +169,34 @@ class JSGraph : public EmbedderGraph {
         return MaybeLocal<Array>();
     }
 
-    {
-      HandleScope handle_scope(isolate_);
-      for (const auto& edge_info : edges_) {
-        Node* source = edge_info.first;
-        Local<Value> edges;
-        if (!info_objects[source]->Get(context, edges_string).ToLocal(&edges) ||
-            !edges->IsArray()) {
-          return MaybeLocal<Array>();
-        }
+    for (const auto& edge_info : edges_) {
+      Node* source = edge_info.first;
+      Local<Value> edges;
+      if (!info_objects[source]->Get(context, edges_string).ToLocal(&edges) ||
+          !edges->IsArray()) {
+        return MaybeLocal<Array>();
+      }
 
-        size_t i = 0;
-        size_t j = 0;
-        for (const auto& edge : edge_info.second) {
-          Local<Object> to_object = info_objects[edge.second];
-          Local<Object> edge_obj = Object::New(isolate_);
-          Local<Value> edge_name_value;
-          const char* edge_name = edge.first;
-          if (edge_name != nullptr) {
-            if (!String::NewFromUtf8(isolate_, edge_name)
-                     .ToLocal(&edge_name_value)) {
-              return MaybeLocal<Array>();
-            }
-          } else {
-            edge_name_value = Number::New(isolate_, static_cast<double>(j++));
-          }
-          if (edge_obj->Set(context, name_string, edge_name_value)
-                  .IsNothing() ||
-              edge_obj->Set(context, to_string, to_object).IsNothing() ||
-              edges.As<Array>()->Set(context, i++, edge_obj).IsNothing()) {
+      size_t i = 0;
+      size_t j = 0;
+      for (const auto& edge : edge_info.second) {
+        HandleScope handle_scope(isolate_);
+        Local<Object> to_object = info_objects[edge.second];
+        Local<Object> edge_obj = Object::New(isolate_);
+        Local<Value> edge_name_value;
+        const char* edge_name = edge.first;
+        if (edge_name != nullptr) {
+          if (!String::NewFromUtf8(isolate_, edge_name)
+                   .ToLocal(&edge_name_value)) {
             return MaybeLocal<Array>();
           }
+        } else {
+          edge_name_value = Number::New(isolate_, static_cast<double>(j++));
+        }
+        if (edge_obj->Set(context, name_string, edge_name_value).IsNothing() ||
+            edge_obj->Set(context, to_string, to_object).IsNothing() ||
+            edges.As<Array>()->Set(context, i++, edge_obj).IsNothing()) {
+          return MaybeLocal<Array>();
         }
       }
     }