From 9cb4de2ea81e8b8285a1fa6d299cfa884f38d947 Mon Sep 17 00:00:00 2001
From: Julien Fontanet <julien.fontanet@isonoe.net>
Date: Mon, 22 Jun 2015 23:33:52 +0200
Subject: [PATCH] Inline an only-used-once function.

---
 packages/xen-api/src/index.js | 55 +++++++++++++++++------------------
 1 file changed, 26 insertions(+), 29 deletions(-)

diff --git a/packages/xen-api/src/index.js b/packages/xen-api/src/index.js
index ae567189d..d032d54bd 100644
--- a/packages/xen-api/src/index.js
+++ b/packages/xen-api/src/index.js
@@ -121,36 +121,10 @@ let getNotConnectedPromise = function () {
   return promise
 }
 
-// ===================================================================
+// -------------------------------------------------------------------
 
 const OPAQUE_REF_RE = /^OpaqueRef:/
 
-function createAutoLinks (collection, object) {
-  forEach(object, function resolveObject (value, key, object) {
-    if (isArray(value)) {
-      // Do not create an array of links unless it is known this is an
-      // array of refs.
-      if (value.length && !OPAQUE_REF_RE.test(value)) {
-        return
-      }
-
-      defineProperty(object, '$' + key, {
-        get () {
-          return map(value, (ref) => collection[ref])
-        }
-      })
-    } else if (isObject(value)) {
-      forEach(value, resolveObject)
-    } else if (OPAQUE_REF_RE.test(value)) {
-      defineProperty(object, '$' + key, {
-        get () {
-          return collection[value]
-        }
-      })
-    }
-  })
-}
-
 // ===================================================================
 
 const MAX_TRIES = 5
@@ -404,9 +378,32 @@ export class Xapi extends EventEmitter {
   _addObject (type, ref, object) {
     const {_objectsByRefs: objectsByRefs} = this
 
-    createAutoLinks(objectsByRefs, object)
+    // Creates resolved properties.
+    forEach(object, function resolveObject (value, key, object) {
+      if (isArray(value)) {
+        // Do not create an array of links unless it is known this is
+        // an array of refs.
+        if (value.length && !OPAQUE_REF_RE.test(value)) {
+          return
+        }
 
-    // All custom properties are non read-only and non enumerable.
+        defineProperty(object, '$' + key, {
+          get () {
+            return map(value, (ref) => objectsByRefs[ref])
+          }
+        })
+      } else if (isObject(value)) {
+        forEach(value, resolveObject)
+      } else if (OPAQUE_REF_RE.test(value)) {
+        defineProperty(object, '$' + key, {
+          get () {
+            return objectsByRefs[value]
+          }
+        })
+      }
+    })
+
+    // All custom properties are read-only and non enumerable.
     defineProperties(object, {
       $id: { value: object.uuid || ref },
       $pool: { get: () => this._pool },