src: use persistent strings from node::Environment

bnoordhuis · MylesBorins · commit 6a6c431eecfe · 2017-03-28T18:28:57.000-04:00
Replace a few calls to FIXED_ONE_BYTE_STRING() with their persistent counterparts from `node::Environment`. None of the calls are in hot code paths but why create a new string when one already exists? PR-URL: #11945 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Brian White <mscdex@mscdex.net>
diff --git a/src/node_contextify.cc b/src/node_contextify.cc
@@ -722,8 +722,7 @@ class ContextifyScript : public BaseObject {
       return -1;
     }
 
-    Local<String> key = FIXED_ONE_BYTE_STRING(env->isolate(), "timeout");
-    Local<Value> value = options.As<Object>()->Get(key);
+    Local<Value> value = options.As<Object>()->Get(env->timeout_string());
     if (value->IsUndefined()) {
       return -1;
     }
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
@@ -2055,8 +2055,7 @@ void SSLWrap<Base>::VerifyError(const FunctionCallbackInfo<Value>& args) {
   Local<String> reason_string = OneByteString(isolate, reason);
   Local<Value> exception_value = Exception::Error(reason_string);
   Local<Object> exception_object = exception_value->ToObject(isolate);
-  exception_object->Set(FIXED_ONE_BYTE_STRING(isolate, "code"),
-                        OneByteString(isolate, code));
+  exception_object->Set(w->env()->code_string(), OneByteString(isolate, code));
   args.GetReturnValue().Set(exception_object);
 }
 
@@ -3255,7 +3254,7 @@ void Connection::SetSNICallback(const FunctionCallbackInfo<Value>& args) {
   }
 
   Local<Object> obj = Object::New(env->isolate());
-  obj->Set(FIXED_ONE_BYTE_STRING(args.GetIsolate(), "onselect"), args[0]);
+  obj->Set(env->onselect_string(), args[0]);
   conn->sniObject_.Reset(args.GetIsolate(), obj);
 }
 #endif
@@ -5583,7 +5582,7 @@ void RandomBytes(const FunctionCallbackInfo<Value>& args) {
   RandomBytesRequest* req = new RandomBytesRequest(env, obj, size);
 
   if (args[1]->IsFunction()) {
-    obj->Set(FIXED_ONE_BYTE_STRING(args.GetIsolate(), "ondone"), args[1]);
+    obj->Set(env->ondone_string(), args[1]);
 
     if (env->in_domain())
       obj->Set(env->domain_string(), env->domain_array()->Get(0));

Original file line number	Diff line number	Diff line change
`@@ -722,8 +722,7 @@ class ContextifyScript : public BaseObject {`
`722`	`722`	`return -1;`
`723`	`723`	`}`
`724`	`724`
`725`		`- Local<String> key = FIXED_ONE_BYTE_STRING(env->isolate(), "timeout");`
`726`		`- Local<Value> value = options.As<Object>()->Get(key);`
	`725`	`+ Local<Value> value = options.As<Object>()->Get(env->timeout_string());`
`727`	`726`	`if (value->IsUndefined()) {`
`728`	`727`	`return -1;`
`729`	`728`	`}`