Issue 1280 - Update v8

include/AdblockPlus/V8ValueHolder.h

Index: include/AdblockPlus/V8ValueHolder.h
===================================================================
--- a/include/AdblockPlus/V8ValueHolder.h
+++ b/include/AdblockPlus/V8ValueHolder.h
@@ -33,17 +33,10 @@
   class V8ValueHolder
   {
   public:
+    typedef typename v8::Persistent<T> V8Persistent;
     V8ValueHolder()
     {
-      reset(0, v8::Persistent<T>());
-    }
-    V8ValueHolder(V8ValueHolder& value)
-    {
-      reset(value.isolate, static_cast<v8::Handle<T> >(value));
-    }
-    V8ValueHolder(v8::Isolate* isolate, v8::Persistent<T> value)
-    {
-      reset(isolate, value);
+      reset();
     }
     V8ValueHolder(v8::Isolate* isolate, v8::Handle<T> value)
     {
@@ -57,7 +50,11 @@
         this->value.reset(0);
       }
     }
-    void reset(v8::Isolate* isolate, v8::Persistent<T> value)
+    void reset(v8::Isolate* isolate, v8::Handle<T> value)
+    {
+        reset(isolate, std::auto_ptr<V8Persistent>(new V8Persistent(isolate, value)));
+    }
+    void reset(v8::Isolate* isolate = nullptr, std::auto_ptr<V8Persistent> value = std::auto_ptr<V8Persistent>(new V8Persistent()))

[Felix Dahlke, 2014/10/31 16:30:15]

nullptr is also C++11, I figure we'd better stick to 0 until we switch.

[sergei, 2014/10/31 17:14:17]

fixed

     {
       if (this->value.get())
       {
@@ -65,33 +62,20 @@
         this->value.reset(0);
       }
 
-      if (!value.IsEmpty())
+      if (!value->IsEmpty())
       {
         this->isolate = isolate;
-        this->value.reset(new v8::Persistent<T>(value));
+        this->value = value;
       }
     }
 
-    void reset(v8::Isolate* isolate, v8::Handle<T> value)
-    {
-      reset(isolate, v8::Persistent<T>::New(isolate, value));
-    }
-
-    T* operator->() const
-    {
-      return **value;
-    }
-    operator v8::Handle<T>() const
-    {
-      return *value;
-    }
-    operator v8::Persistent<T>() const
+    operator V8Persistent&() const
     {
       return *value;
     }
   private:
     v8::Isolate* isolate;
-    std::auto_ptr<v8::Persistent<T> > value;
+    std::auto_ptr<V8Persistent> value;
   };
 }