diff options
author | Ben Noordhuis <info@bnoordhuis.nl> | 2013-07-03 04:23:44 +0200 |
---|---|---|
committer | Ben Noordhuis <info@bnoordhuis.nl> | 2013-07-06 17:44:44 +0200 |
commit | 110a9cd8db515c4d1a9ac5cd8837291da7c6c5ea (patch) | |
tree | 71e5a14a98131d89d670f842eb36bfcccab00b7b /src/node_wrap.h | |
parent | 9b3de60d3537df657e75887436a5b1df5ed80c2d (diff) | |
download | node-new-110a9cd8db515c4d1a9ac5cd8837291da7c6c5ea.tar.gz |
lib, src: upgrade after v8 api change
This is a big commit that touches just about every file in the src/
directory. The V8 API has changed in significant ways. The most
important changes are:
* Binding functions take a const v8::FunctionCallbackInfo<T>& argument
rather than a const v8::Arguments& argument.
* Binding functions return void rather than v8::Handle<v8::Value>. The
return value is returned with the args.GetReturnValue().Set() family
of functions.
* v8::Persistent<T> no longer derives from v8::Handle<T> and no longer
allows you to directly dereference the object that the persistent
handle points to. This means that the common pattern of caching
oft-used JS values in a persistent handle no longer quite works,
you first need to reconstruct a v8::Local<T> from the persistent
handle with the Local<T>::New(isolate, persistent) factory method.
A handful of (internal) convenience classes and functions have been
added to make dealing with the new API a little easier.
The most visible one is node::Cached<T>, which wraps a v8::Persistent<T>
with some template sugar. It can hold arbitrary types but so far it's
exclusively used for v8::Strings (which was by far the most commonly
cached handle type.)
Diffstat (limited to 'src/node_wrap.h')
-rw-r--r-- | src/node_wrap.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/node_wrap.h b/src/node_wrap.h index 1b7b451627..ff7179fdb8 100644 --- a/src/node_wrap.h +++ b/src/node_wrap.h @@ -39,15 +39,15 @@ extern v8::Persistent<v8::FunctionTemplate> tcpConstructorTmpl; #define WITH_GENERIC_STREAM(obj, BODY) \ do { \ if (!tcpConstructorTmpl.IsEmpty() && \ - tcpConstructorTmpl->HasInstance(obj)) { \ + HasInstance(tcpConstructorTmpl, obj)) { \ PipeWrap* wrap = PipeWrap::Unwrap(obj); \ BODY \ } else if (!ttyConstructorTmpl.IsEmpty() && \ - ttyConstructorTmpl->HasInstance(obj)) { \ + HasInstance(ttyConstructorTmpl, obj)) { \ TTYWrap* wrap = TTYWrap::Unwrap(obj); \ BODY \ } else if (!pipeConstructorTmpl.IsEmpty() && \ - pipeConstructorTmpl->HasInstance(obj)) { \ + HasInstance(pipeConstructorTmpl, obj)) { \ TCPWrap* wrap = TCPWrap::Unwrap(obj); \ BODY \ } \ |