summaryrefslogtreecommitdiff
path: root/Makefile
diff options
context:
space:
mode:
authorJoyee Cheung <joyeec9h3@gmail.com>2019-06-09 21:04:51 +0800
committerJoyee Cheung <joyeec9h3@gmail.com>2019-06-17 10:18:09 +0800
commit1432065e9dc77218507f328b63cb7f5d279dd6e9 (patch)
tree0f4273124106700d9069281b6588b64a4ccf2b2d /Makefile
parent28d3f1963a98ce157b360c0a3ba70fb8e1f73f60 (diff)
downloadandroid-node-v8-1432065e9dc77218507f328b63cb7f5d279dd6e9.tar.gz
android-node-v8-1432065e9dc77218507f328b63cb7f5d279dd6e9.tar.bz2
android-node-v8-1432065e9dc77218507f328b63cb7f5d279dd6e9.zip
lib: correct error.errno to always be numeric
Historically `error.errno` of system errors thrown by Node.js can sometimes be the same as `err.code`, which are string representations of the error numbers. This is useless and incorrect, and results in an information loss for users since then they will have to resort to something like `process.binding('uv'[`UV_${errno}`])` to get to the numeric error codes. This patch corrects this behavior by always setting `error.errno` to be negative numbers. For fabricated errors like `ENOTFOUND`, `error.errno` is now undefined since there is no numeric equivalent for them anyway. For c-ares errors, `error.errno` is now undefined because the numeric representations (negated) can be in conflict with libuv error codes - this is fine since numeric codes was not available for c-ares errors anyway. Users can use the public API `util.getSystemErrorName(errno)` to retrieve string codes for these numbers. PR-URL: https://github.com/nodejs/node/pull/28140 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Diffstat (limited to 'Makefile')
0 files changed, 0 insertions, 0 deletions