Skip to content

n-api: use nullptr instead of NULL in node_api.cc

Rodrigo Muino Tomonari requested to merge github/fork/danbev/node_api_null_usage into master

This commit changes two checks which use NULL to use nullptr.

I'm not very familiar with N-API but wanted to bring this up in case it was something that was overlooked.

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

n-api

cc @nodejs/api

Merge request reports