Skip to content

Commit

Permalink
fix: make node detection a bit more forgiving (#1445)
Browse files Browse the repository at this point in the history
* Make node detection a bit more forgiving

* reorder

* make sure we get a boolean
  • Loading branch information
dcodeIO authored Jul 15, 2020
1 parent 07e8185 commit 4e75f6d
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 13 deletions.
9 changes: 6 additions & 3 deletions index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1951,15 +1951,18 @@ export namespace util {
public length(): number;
}

/** Whether running within node or not. */
let isNode: boolean;

/** Global object reference. */
let global: object;

/** An immuable empty array. */
const emptyArray: any[];

/** An immutable empty object. */
const emptyObject: object;

/** Whether running within node or not. */
const isNode: boolean;

/**
* Tests if the specified value is an integer.
* @param value Value to test
Expand Down
27 changes: 17 additions & 10 deletions src/util/minimal.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,23 @@ util.pool = require("@protobufjs/pool");
// utility to work with the low and high bits of a 64 bit value
util.LongBits = require("./longbits");

// global object reference
util.global = typeof global !== "undefined" && Object.prototype.toString.call(global) === "[object global]" && global
/**
* Whether running within node or not.
* @memberof util
* @type {boolean}
*/
util.isNode = Boolean(typeof global !== "undefined"
&& global
&& global.process
&& global.process.versions
&& global.process.versions.node);

/**
* Global object reference.
* @memberof util
* @type {Object}
*/
util.global = util.isNode && global
|| typeof window !== "undefined" && window
|| typeof self !== "undefined" && self
|| this; // eslint-disable-line no-invalid-this
Expand All @@ -46,14 +61,6 @@ util.emptyArray = Object.freeze ? Object.freeze([]) : /* istanbul ignore next */
*/
util.emptyObject = Object.freeze ? Object.freeze({}) : /* istanbul ignore next */ {}; // used on prototypes

/**
* Whether running within node or not.
* @memberof util
* @type {boolean}
* @const
*/
util.isNode = Boolean(util.global.process && util.global.process.versions && util.global.process.versions.node);

/**
* Tests if the specified value is an integer.
* @function
Expand Down

0 comments on commit 4e75f6d

Please sign in to comment.