ffi: add new methods and error codes#62762
Conversation
|
Review requested:
|
|
Great! Thank you. It looks like I won't need my own unsafe-pointer once node:ffi gets stabilized 🎉 😄 |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #62762 +/- ##
==========================================
- Coverage 89.69% 89.59% -0.10%
==========================================
Files 706 706
Lines 218143 218147 +4
Branches 41730 41687 -43
==========================================
- Hits 195655 195459 -196
- Misses 14401 14586 +185
- Partials 8087 8102 +15
🚀 New features to boost your workflow:
|
|
LGTM |
|
@meixg I don't think it is acually necessary. When using the function you already own the source ( |
927da3b to
3e3b1da
Compare
Signed-off-by: Paolo Insogna <paolo@cowtech.it>
3e3b1da to
1fee1dd
Compare
benchmark/ffi/getpid.jsusingffi.dlopen(null, ...)+uv_os_getpid. Thanks to @bengl for Foreign Function Interface (FFI) implementation #46905doc/api/errors.mdas asked from @jasnell in lib,src,test,doc: add node:ffi module #62072 (comment)As promised to @justjake, also the following followups requested in #62072 (comment):
ffi.exportArrayBuffer(...)andffi.exportArrayBufferView(...).exportBytes) to avoid extra JS Buffer wrapper allocations.