Make JsEngine::Evaluate() return a wrapper for v8::Value to accessdifferent variable types easily

include/AdblockPlus/JsValue.h

 #ifndef ADBLOCKPLUS_JS_VALUE_H
 #define ADBLOCKPLUS_JS_VALUE_H
 
 #include <vector>
 #include <v8.h>
 #ifdef _MSC_VER
 #include <memory>
 #else
 #include <tr1/memory>
 #endif
@@ skipping 28 matching lines @@
     JsValuePtr GetProperty(const std::string& name) const;
     void SetProperty(const std::string& name, const std::string& val);
     void SetProperty(const std::string& name, int64_t val);
     void SetProperty(const std::string& name, bool val);
     inline void SetProperty(const std::string& name, const char* val)
     {
       SetProperty(name, std::string(val));
     }
     inline void SetProperty(const std::string& name, int val)
     {
-      SetProperty(name, (int64_t)val);

[Felix Dahlke, 2013/04/16 15:14:33]

As above, I'd prefer static_cast.

+      SetProperty(name, static_cast<int64_t>(val));
     }
     std::string GetClassName() const;
     JsValuePtr Call(const JsValueList& params = JsValueList(),
         AdblockPlus::JsValuePtr thisPtr = AdblockPlus::JsValuePtr()) const;
   protected:
     JsValue(JsEngine& jsEngine, v8::Handle<v8::Value> value);
     void SetProperty(const std::string& name, v8::Handle<v8::Value> val);
 
     JsEngine& jsEngine;
     v8::Persistent<v8::Value> value;
   };
 }
 
 #endif