Sitelet https://web.archive.org/web/20201231110236/https://github.com/nodejs/node-addon-examples/pull/151
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove wrapper_ from napi_wrap examples #151

Open
wants to merge 3 commits into
base: master
from

Conversation

@gabrielschulhof
Copy link
Contributor

@gabrielschulhof gabrielschulhof commented Aug 4, 2020

The variable is only ever assigned and then cleaned up, never used.

Signed-off-by: Gabriel Schulhof <gabriel.schulhof@intel.com>
Fixes: #140
@gabrielschulhof gabrielschulhof force-pushed the gabrielschulhof:remove-wrapper_ branch from 904b9b2 to 5c94d85 Aug 5, 2020
@mhdawson
Copy link
Member

@mhdawson mhdawson commented Aug 18, 2020

I'm wondering if it it still makes sense for an example to show how you can use one of the parameters even if its not really fully used. @gabrielschulhof what's your take on that?

@mhdawson
Copy link
Member

@mhdawson mhdawson commented Sep 22, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.