search

nodejs / node-api-headers

Repository used to make the N-API headers more accessible
MIT License
32 stars 13 forks source link

Wrong symbols in v6.node_api_symbols #5

Closed guentner closed 1 year ago

guentner commented 1 year ago

v6.node_api_symbols should probably extend v5.node_api_symbols instead of v5.js_native_api_symbols.

https://github.com/nodejs/node-api-headers/blob/d68505e4055ecb630e14c26c32e5c2c65e179bba/symbols.js#L189

NickNaso commented 1 year ago

@guentner you are right I just made a PR to solve the problem. Thanks for reporting.

legendecas commented 1 year ago

Fixed in #6