Skip to content

n-api: use Maybe version of SetPrototype

Rodrigo Muino Tomonari requested to merge github/fork/danbev/napi-setprototype-warning into master

Currently the following two warnings are displayed when compiling:

../src/node_api.cc:1966:12: warning: 'SetPrototype' is deprecated
[-Wdeprecated-declarations]
  wrapper->SetPrototype(proto);
           ^

../src/node_api.cc:1967:8: warning: 'SetPrototype' is deprecated
[-Wdeprecated-declarations]
  obj->SetPrototype(wrapper);

This commit changes these calls to use the Maybe version.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

n-api

Merge request reports