Skip to content

n-api: allow escape of undefined

Rodrigo Muino Tomonari requested to merge escape_undefined into master

The node-addon-api module was calling escape on undefined which would fail. Instead of forcing a check in all consumers of napi_escape_handle accept undefined and simply return it as it does not need to be escaped.

Refs: https://github.com/nodejs/node-addon-api/issues/233

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

Merge request reports