util: move getConstructorOf() to internal

PR-URL: #12526 Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Apr 24, 2017 · 3c0dd45 · 3c0dd45
1 parent b2870a4
commit 3c0dd45
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 17 deletions.
diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -180,6 +180,21 @@ function convertToValidSignal(signal) {
   throw new Error('Unknown signal: ' + signal);
 }
 
+function getConstructorOf(obj) {
+  while (obj) {
+    var descriptor = Object.getOwnPropertyDescriptor(obj, 'constructor');
+    if (descriptor !== undefined &&
+        typeof descriptor.value === 'function' &&
+        descriptor.value.name !== '') {
+      return descriptor.value;
+    }
+
+    obj = Object.getPrototypeOf(obj);
+  }
+
+  return null;
+}
+
 module.exports = exports = {
   assertCrypto,
   cachedResult,
@@ -188,6 +203,7 @@ module.exports = exports = {
   decorateErrorStack,
   deprecate,
   filterDuplicateStrings,
+  getConstructorOf,
   isError,
   normalizeEncoding,
   objectToString,

diff --git a/lib/util.js b/lib/util.js
@@ -278,22 +278,6 @@ function arrayToHash(array) {
 }
 
 
-function getConstructorOf(obj) {
-  while (obj) {
-    var descriptor = Object.getOwnPropertyDescriptor(obj, 'constructor');
-    if (descriptor !== undefined &&
-        typeof descriptor.value === 'function' &&
-        descriptor.value.name !== '') {
-      return descriptor.value;
-    }
-
-    obj = Object.getPrototypeOf(obj);
-  }
-
-  return null;
-}
-
-
 function ensureDebugIsInitialized() {
   if (Debug === undefined) {
     const runInDebugContext = require('vm').runInDebugContext;
@@ -410,7 +394,7 @@ function formatValue(ctx, value, recurseTimes) {
     });
   }
 
-  var constructor = getConstructorOf(value);
+  var constructor = internalUtil.getConstructorOf(value);
 
   // Some type of object without properties can be shortcutted.
   if (keys.length === 0) {