search

v8 / node

Node.js JavaScript runtime :sparkles::turtle::rocket::sparkles:
https://nodejs.org
Other
171 stars 66 forks source link

Don't use WrapperDescriptor and instead use Wrap/Unwrap APIs #187

Closed mlippautz closed 5 months ago

mlippautz commented 5 months ago

Node has very basic support for wrapping objects and an experimental API to use CppHeap (Oilpan).

This PR switches the API to using v8::Object::Wrap() and v8::Object::Unwrap() instead of using regular embedder fields.

The PR is likely incomplete but enough to get basic tests to compile.

Also, the old apporach did distinguish between managed and unmanaged wrappers. Really, this distinction is gone as the new Wrap/Unrap APIs only support a single (the current) CppHeap.