mirror of https://github.com/nodejs/node.git
net: refactor check for Windows
PR-URL: https://github.com/nodejs/node/pull/33497 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>pull/33529/head
parent
dc6c93c03e
commit
c5719eb265
11
lib/net.js
11
lib/net.js
|
@ -119,6 +119,8 @@ const { kTimeout } = require('internal/timers');
|
|||
const DEFAULT_IPV4_ADDR = '0.0.0.0';
|
||||
const DEFAULT_IPV6_ADDR = '::';
|
||||
|
||||
const isWindows = process.platform === 'win32';
|
||||
|
||||
function noop() {}
|
||||
|
||||
function getFlags(ipv6Only) {
|
||||
|
@ -330,8 +332,7 @@ function Socket(options) {
|
|||
this[async_id_symbol] = this._handle.getAsyncId();
|
||||
|
||||
if ((fd === 1 || fd === 2) &&
|
||||
(this._handle instanceof Pipe) &&
|
||||
process.platform === 'win32') {
|
||||
(this._handle instanceof Pipe) && isWindows) {
|
||||
// Make stdout and stderr blocking on Windows
|
||||
err = this._handle.setBlocking(true);
|
||||
if (err)
|
||||
|
@ -1004,7 +1005,7 @@ function lookupAndConnect(self, options) {
|
|||
hints: options.hints || 0
|
||||
};
|
||||
|
||||
if (process.platform !== 'win32' &&
|
||||
if (!isWindows &&
|
||||
dnsopts.family !== 4 &&
|
||||
dnsopts.family !== 6 &&
|
||||
dnsopts.hints === 0) {
|
||||
|
@ -1207,7 +1208,7 @@ function createServerHandle(address, port, addressType, fd, flags) {
|
|||
assert(!address && !port);
|
||||
} else if (port === -1 && addressType === -1) {
|
||||
handle = new Pipe(PipeConstants.SERVER);
|
||||
if (process.platform === 'win32') {
|
||||
if (isWindows) {
|
||||
const instances = parseInt(process.env.NODE_PENDING_PIPE_INSTANCES);
|
||||
if (!NumberIsNaN(instances)) {
|
||||
handle.setPendingInstances(instances);
|
||||
|
@ -1690,7 +1691,7 @@ Server.prototype.unref = function() {
|
|||
let _setSimultaneousAccepts;
|
||||
let warnSimultaneousAccepts = true;
|
||||
|
||||
if (process.platform === 'win32') {
|
||||
if (isWindows) {
|
||||
let simultaneousAccepts;
|
||||
|
||||
_setSimultaneousAccepts = function(handle) {
|
||||
|
|
Loading…
Reference in New Issue