From 4d6b8c76a73297fa845a7b195daa4947a71d28f0 Mon Sep 17 00:00:00 2001
From: Benjamin Byholm <bbyholm@abo.fi>
Date: Tue, 13 Oct 2015 00:42:26 +0300
Subject: [PATCH] Improve performance for Globals

---
 nan_persistent_12_inl.h     | 23 ++++++++++++-----------
 nan_persistent_pre_12_inl.h |  8 ++++----
 2 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/nan_persistent_12_inl.h b/nan_persistent_12_inl.h
index 7779bdcc..1c26f85b 100644
--- a/nan_persistent_12_inl.h
+++ b/nan_persistent_12_inl.h
@@ -84,8 +84,9 @@ template<typename T> class PersistentBase {
  private:
   NAN_INLINE PersistentBase() :
       persistent() {}
-  NAN_INLINE explicit PersistentBase(v8::Persistent<T> that) :
-      persistent(v8::Isolate::GetCurrent(), that) { }
+  NAN_INLINE explicit PersistentBase(v8::Persistent<T> that) {
+    std::memcpy(&that, &persistent, sizeof (v8::Persistent<T>));
+  }
   NAN_INLINE explicit PersistentBase(v8::Local<T> that) :
       persistent(v8::Isolate::GetCurrent(), that) { }
   template<typename S, typename M> friend class Persistent;
@@ -181,12 +182,12 @@ class Global : public PersistentBase<T> {
 
   template <class S>
   NAN_INLINE Global(const PersistentBase<S>& that)
-      : PersistentBase<T>(that) {
+      : PersistentBase<T>(v8::Persistent<T>(v8::Isolate::GetCurrent(), that.persistent)) {
     TYPE_CHECK(T, S);
   }
 
   NAN_INLINE Global(Global&& other) : PersistentBase<T>(other.persistent) {
-    other.Reset();
+    other.Empty();
   }
 
   NAN_INLINE ~Global() { this->Reset(); }
@@ -195,8 +196,8 @@ class Global : public PersistentBase<T> {
   NAN_INLINE Global& operator=(Global<S>&& rhs) {
     TYPE_CHECK(T, S);
      if (this != &rhs) {
-      this->Reset(rhs.persistent);
-      rhs.Reset();
+      std::memcpy(&rhs.persistent, &this->persistent, sizeof (v8::PersistentBase<S>));
+      rhs.Empty();
     }
     return *this;
   }
@@ -226,7 +227,7 @@ class Global : public PersistentBase<T> {
 
   template <typename S>
   NAN_INLINE Global(const PersistentBase<S> &that)
-    : PersistentBase<T>(that) {
+    : PersistentBase<T>(v8::Persistent<T>(v8::Isolate::GetCurrent(), that.persistent)) {
     TYPE_CHECK(T, S);
   }
 
@@ -234,8 +235,8 @@ class Global : public PersistentBase<T> {
    * Move constructor.
    */
   NAN_INLINE Global(RValue rvalue)
-    : PersistentBase<T>(rvalue.object.persistent) {
-    rvalue.object->Reset();
+    : PersistentBase<T>(rvalue.object->persistent) {
+    rvalue.object->Empty();
   }
 
   NAN_INLINE ~Global() { this->Reset(); }
@@ -246,8 +247,8 @@ class Global : public PersistentBase<T> {
   template<typename S>
   NAN_INLINE Global &operator=(Global<S> rhs) {
     TYPE_CHECK(T, S);
-    this->Reset(rhs.persistent);
-    rhs.Reset();
+    std::memcpy(&rhs.persistent, &this->persistent, sizeof (v8::Persistent<T>));
+    rhs.Empty();
     return *this;
   }
 
diff --git a/nan_persistent_pre_12_inl.h b/nan_persistent_pre_12_inl.h
index 2b212159..a5692809 100644
--- a/nan_persistent_pre_12_inl.h
+++ b/nan_persistent_pre_12_inl.h
@@ -207,7 +207,7 @@ class Global : public PersistentBase<T> {
 
   template <typename S>
   NAN_INLINE Global(const PersistentBase<S> &that)
-    : PersistentBase<T>(that) {
+    : PersistentBase<T>(v8::Peristent<T>::New(that.persistent)) {
     TYPE_CHECK(T, S);
   }
   /**
@@ -215,7 +215,7 @@ class Global : public PersistentBase<T> {
    */
   NAN_INLINE Global(RValue rvalue)
     : PersistentBase<T>(rvalue.object.persistent) {
-    rvalue.object->Reset();
+    rvalue.object->Clear();
   }
   NAN_INLINE ~Global() { this->Reset(); }
   /**
@@ -224,8 +224,8 @@ class Global : public PersistentBase<T> {
   template<typename S>
   NAN_INLINE Global &operator=(Global<S> rhs) {
     TYPE_CHECK(T, S);
-    this->Reset(rhs.persistent);
-    rhs.Reset();
+    this->persistent = rhs.persistent;
+    rhs.Clear();
     return *this;
   }
   /**