domain: do not import util for a simple type check

This removes `require('util')` from the `domain` module. There was
only a single simple type check used from the `util` module which
is now inlined instead.

PR-URL: https://github.com/nodejs/node/pull/29825
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
This commit is contained in:
Ruben Bridgewater 2019-10-03 02:56:18 +02:00 committed by Rich Trott
parent 27507ca108
commit fce1a5198a
1 changed files with 1 additions and 2 deletions

View File

@ -28,7 +28,6 @@
const { Object, Reflect } = primordials;
const util = require('util');
const EventEmitter = require('events');
const {
ERR_DOMAIN_CALLBACK_NOT_AVAILABLE,
@ -207,7 +206,7 @@ Domain.prototype.members = undefined;
Domain.prototype._errorHandler = function(er) {
var caught = false;
if (!util.isPrimitive(er)) {
if ((typeof er === 'object' && er !== null) || typeof er === 'function') {
Object.defineProperty(er, 'domain', {
configurable: true,
enumerable: false,