tls: refactor tls_wrap.cc

Store the result of excetuting the function in variable. Instead of excetuting it for multiple times. PR-URL: nodejs#30303 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
Trott · Nov 8, 2019 · 632cadd · 632cadd
1 parent 55f4db1
commit 632cadd
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/tls_wrap.cc b/src/tls_wrap.cc
@@ -1141,12 +1141,11 @@ void TLSWrap::Initialize(Local<Object> target,
   env->SetProtoMethod(t, "getServername", GetServername);
   env->SetProtoMethod(t, "setServername", SetServername);
 
-  env->set_tls_wrap_constructor_function(
-      t->GetFunction(env->context()).ToLocalChecked());
+  Local<Function> fn = t->GetFunction(env->context()).ToLocalChecked();
 
-  target->Set(env->context(),
-              tlsWrapString,
-              t->GetFunction(env->context()).ToLocalChecked()).Check();
+  env->set_tls_wrap_constructor_function(fn);
+
+  target->Set(env->context(), tlsWrapString, fn).Check();
 }
 
 }  // namespace node