diff --git a/Lib/javascript/v8/javascriptcode.swg b/Lib/javascript/v8/javascriptcode.swg index 39c1f3a79..ebe841b22 100644 --- a/Lib/javascript/v8/javascriptcode.swg +++ b/Lib/javascript/v8/javascriptcode.swg @@ -7,19 +7,18 @@ * - $jsmangledtype: mangled type of class * ----------------------------------------------------------------------------- */ %fragment("js_ctor", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments& args) { +void $jswrapper(const v8::FunctionCallbackInfo& args) { v8::HandleScope scope; v8::Handle self = args.Holder(); $jslocals if(args.Length() != $jsargcount) SWIG_exception_fail(SWIG_ERROR, "Illegal number of arguments for $jswrapper."); $jscode - - SWIGV8_SetPrivateData(self, result, SWIGTYPE_$jsmangledtype, SWIG_POINTER_OWN); - return scope.Close(self); - goto fail; + SWIGV8_SetPrivateData(self, result, SWIGTYPE_$jsmangledtype, SWIG_POINTER_OWN); + args.GetReturnValue().Set(self); + fail: - return scope.Close(v8::Undefined()); + return; } %} @@ -30,10 +29,9 @@ fail: * ----------------------------------------------------------------------------- */ %fragment ("js_veto_ctor", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments& args) { +void $jswrapper(const v8::FunctionCallbackInfo& args) { v8::HandleScope scope; SWIG_exception(SWIG_ERROR, "Class $jsname can not be instantiated"); - return scope.Close(v8::Undefined()); } %} @@ -45,7 +43,7 @@ v8::Handle $jswrapper(const v8::Arguments& args) { * ----------------------------------------------------------------------------- */ %fragment ("js_ctor_dispatcher", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments& args) { +void $jswrapper(const v8::FunctionCallbackInfo& args) { v8::HandleScope scope; OverloadErrorHandler errorHandler; v8::Handle self; @@ -57,7 +55,7 @@ v8::Handle $jswrapper(const v8::Arguments& args) { SWIG_exception_fail(SWIG_ERROR, "Illegal arguments for contruction of $jsmangledname"); fail: - return scope.Close(v8::Undefined()); + return; } %} @@ -70,7 +68,7 @@ fail: * - $jsmangledtype: mangled type of class * ----------------------------------------------------------------------------- */ %fragment("js_overloaded_ctor", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments& args, V8ErrorHandler& SWIGV8_ErrorHandler) { +void $jswrapper(const v8::FunctionCallbackInfo& args, V8ErrorHandler& SWIGV8_ErrorHandler) { v8::HandleScope scope; v8::Handle self = args.Holder(); $jslocals @@ -78,11 +76,11 @@ v8::Handle $jswrapper(const v8::Arguments& args, V8ErrorHandler& SWIG $jscode SWIGV8_SetPrivateData(self, result, SWIGTYPE_$jsmangledtype, SWIG_POINTER_OWN); - return scope.Close(self); + args.GetReturnValue().Set(self); goto fail; fail: - return scope.Close(v8::Undefined()); + return; } %} @@ -97,10 +95,9 @@ fail: %{ if(args.Length() == $jsargcount) { errorHandler.err.Clear(); - self = $jswrapper(args, errorHandler); - if(errorHandler.err.IsEmpty()) { - return scope.Close(self); - } + $jswrapper(args, errorHandler); + if(errorHandler.err.IsEmpty()) + return; } %} @@ -171,15 +168,16 @@ void $jswrapper(v8::Persistent< v8::Value > object, void *parameter) { * ----------------------------------------------------------------------------- */ %fragment("js_getter", "templates") %{ -v8::Handle $jswrapper(v8::Local property, const v8::AccessorInfo& info) { +void $jswrapper(v8::Local property, const v8::PropertyCallbackInfo& info) { v8::HandleScope scope; v8::Handle jsresult; $jslocals $jscode - return scope.Close(jsresult); + info.GetReturnValue().Set(jsresult); + return; goto fail; fail: - return scope.Close(v8::Undefined()); + return; } %} @@ -191,7 +189,7 @@ fail: * ----------------------------------------------------------------------------- */ %fragment("js_setter", "templates") %{ -void $jswrapper(v8::Local property, v8::Local value, const v8::AccessorInfo& info) { +void $jswrapper(v8::Local property, v8::Local value, const v8::PropertyCallbackInfo& info) { v8::HandleScope scope; $jslocals $jscode @@ -209,17 +207,18 @@ fail: * ----------------------------------------------------------------------------- */ %fragment("js_function", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments &args) { +void $jswrapper(const v8::FunctionCallbackInfo& args) { v8::HandleScope scope; v8::Handle jsresult; $jslocals if(args.Length() != $jsargcount) SWIG_exception_fail(SWIG_ERROR, "Illegal number of arguments for $jswrapper."); - + $jscode - return scope.Close(jsresult); + args.GetReturnValue().Set(jsresult); + return; goto fail; fail: - return scope.Close(v8::Undefined()); + return; } %} @@ -232,7 +231,7 @@ fail: * ----------------------------------------------------------------------------- */ %fragment("js_function_dispatcher", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments &args) { +void $jswrapper(const v8::FunctionCallbackInfo& args) { v8::HandleScope scope; v8::Handle jsresult; OverloadErrorHandler errorHandler; @@ -242,7 +241,7 @@ v8::Handle $jswrapper(const v8::Arguments &args) { goto fail; fail: - return scope.Close(v8::Undefined()); + return; } %} @@ -254,16 +253,17 @@ fail: * ----------------------------------------------------------------------------- */ %fragment ("js_overloaded_function", "templates") %{ -v8::Handle $jswrapper(const v8::Arguments &args, V8ErrorHandler& SWIGV8_ErrorHandler) +void $jswrapper(const v8::FunctionCallbackInfo& args, V8ErrorHandler& SWIGV8_ErrorHandler) { v8::HandleScope scope; v8::Handle jsresult; $jslocals $jscode - return scope.Close(jsresult); + args.GetReturnValue().Set(jsresult); + return; goto fail; fail: - return scope.Close(jsresult); + return; } %} @@ -277,10 +277,9 @@ fail: %{ if(args.Length() == $jsargcount) { errorHandler.err.Clear(); - jsresult = $jswrapper(args, errorHandler); - if(errorHandler.err.IsEmpty()) { - return scope.Close(jsresult); - } + $jswrapper(args, errorHandler); + if(errorHandler.err.IsEmpty()) + return; } %} diff --git a/Lib/javascript/v8/javascripthelpers.swg b/Lib/javascript/v8/javascripthelpers.swg index d28243d88..847e9b73c 100644 --- a/Lib/javascript/v8/javascripthelpers.swg +++ b/Lib/javascript/v8/javascripthelpers.swg @@ -17,7 +17,7 @@ v8::Handle SWIGV8_CreateClassTemplate(const char* symbol) /** * Registers a class method with given name for a given class template. */ -void SWIGV8_AddMemberFunction(v8::Handle class_templ, const char* symbol, v8::InvocationCallback _func) { +void SWIGV8_AddMemberFunction(v8::Handle class_templ, const char* symbol, v8::FunctionCallback _func) { v8::Handle proto_templ = class_templ->PrototypeTemplate(); proto_templ->Set(v8::String::NewSymbol(symbol), v8::FunctionTemplate::New(_func)); } @@ -25,7 +25,7 @@ void SWIGV8_AddMemberFunction(v8::Handle class_templ, cons /** * Registers a class property with given name for a given class template. */ -void SWIGV8_AddMemberVariable(v8::Handle class_templ, const char* symbol, v8::AccessorGetter getter, v8::AccessorSetter setter) { +void SWIGV8_AddMemberVariable(v8::Handle class_templ, const char* symbol, v8::AccessorGetterCallback getter, v8::AccessorSetterCallback setter) { v8::Handle proto_templ = class_templ->InstanceTemplate(); proto_templ->SetAccessor(v8::String::NewSymbol(symbol), getter, setter); } @@ -33,18 +33,18 @@ void SWIGV8_AddMemberVariable(v8::Handle class_templ, cons /** * Registers a class method with given name for a given object. */ -void SWIGV8_AddStaticFunction(v8::Handle obj, const char* symbol, v8::InvocationCallback _func) { +void SWIGV8_AddStaticFunction(v8::Handle obj, const char* symbol, const v8::FunctionCallback& _func) { obj->Set(v8::String::NewSymbol(symbol), v8::FunctionTemplate::New(_func)->GetFunction()); } /** * Registers a class method with given name for a given object. */ -void SWIGV8_AddStaticVariable(v8::Handle obj, const char* symbol, v8::AccessorGetter getter, v8::AccessorSetter setter) { +void SWIGV8_AddStaticVariable(v8::Handle obj, const char* symbol, v8::AccessorGetterCallback getter, v8::AccessorSetterCallback setter) { obj->SetAccessor(v8::String::NewSymbol(symbol), getter, setter); } -void JS_veto_set_variable(v8::Local property, v8::Local value, const v8::AccessorInfo& info) +void JS_veto_set_variable(v8::Local property, v8::Local value, const v8::PropertyCallbackInfo& info) { char buffer[256]; char msg[512];