lib,src: fix consistent spacing inside braces

PR-URL: nodejs#14162 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
jasnell · Sep 20, 2017 · cfb0d73 · cfb0d73
1 parent bc8bd87
commit cfb0d73
Show file tree

Hide file tree

Showing 261 changed files with 1,000 additions and 964 deletions.
diff --git a/benchmark/_test-double-benchmarker.js b/benchmark/_test-double-benchmarker.js
@@ -3,5 +3,5 @@
 const http = require('http');
 
 http.get(process.env.path, function() {
- console.log(JSON.stringify({throughput: 1}));
+ console.log(JSON.stringify({ throughput: 1 }));
 });
diff --git a/benchmark/crypto/aes-gcm-throughput.js b/benchmark/crypto/aes-gcm-throughput.js
@@ -1,7 +1,7 @@
 'use strict';
 var common = require('../common.js');
 var crypto = require('crypto');
-var keylen = {'aes-128-gcm': 16, 'aes-192-gcm': 24, 'aes-256-gcm': 32};
+var keylen = { 'aes-128-gcm': 16, 'aes-192-gcm': 24, 'aes-256-gcm': 32 };
 var bench = common.createBenchmark(main, {
  n: [500],
  cipher: ['aes-128-gcm', 'aes-192-gcm', 'aes-256-gcm'],

diff --git a/benchmark/events/ee-add-remove.js b/benchmark/events/ee-add-remove.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var events = require('events');
 
-var bench = common.createBenchmark(main, {n: [25e4]});
+var bench = common.createBenchmark(main, { n: [25e4] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/events/ee-emit-multi-args.js b/benchmark/events/ee-emit-multi-args.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var EventEmitter = require('events').EventEmitter;
 
-var bench = common.createBenchmark(main, {n: [2e6]});
+var bench = common.createBenchmark(main, { n: [2e6] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/events/ee-emit.js b/benchmark/events/ee-emit.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var EventEmitter = require('events').EventEmitter;
 
-var bench = common.createBenchmark(main, {n: [2e6]});
+var bench = common.createBenchmark(main, { n: [2e6] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/events/ee-listener-count-on-prototype.js b/benchmark/events/ee-listener-count-on-prototype.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var EventEmitter = require('events').EventEmitter;
 
-var bench = common.createBenchmark(main, {n: [5e7]});
+var bench = common.createBenchmark(main, { n: [5e7] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/events/ee-listeners-many.js b/benchmark/events/ee-listeners-many.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var EventEmitter = require('events').EventEmitter;
 
-var bench = common.createBenchmark(main, {n: [5e6]});
+var bench = common.createBenchmark(main, { n: [5e6] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/events/ee-listeners.js b/benchmark/events/ee-listeners.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var EventEmitter = require('events').EventEmitter;
 
-var bench = common.createBenchmark(main, {n: [5e6]});
+var bench = common.createBenchmark(main, { n: [5e6] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/events/ee-once.js b/benchmark/events/ee-once.js
@@ -2,7 +2,7 @@
 var common = require('../common.js');
 var EventEmitter = require('events').EventEmitter;
 
-var bench = common.createBenchmark(main, {n: [2e7]});
+var bench = common.createBenchmark(main, { n: [2e7] });
 
 function main(conf) {
  var n = conf.n | 0;

diff --git a/benchmark/url/url-format.js b/benchmark/url/url-format.js
@@ -3,8 +3,8 @@ const common = require('../common.js');
 const url = require('url');
 
 const inputs = {
- slashes: {slashes: true, host: 'localhost'},
- file: {protocol: 'file:', pathname: '/foo'},
+ slashes: { slashes: true, host: 'localhost' },
+ file: { protocol: 'file:', pathname: '/foo' },
 };
 
 const bench = common.createBenchmark(main, {

diff --git a/benchmark/util/format.js b/benchmark/util/format.js
@@ -17,7 +17,7 @@ const bench = common.createBenchmark(main, {
 const inputs = {
  'string': ['Hello, my name is %s', 'fred'],
  'number': ['Hi, I was born in %d', 1942],
- 'object': ['An error occurred %j', {msg: 'This is an error', code: 'ERR'}],
+ 'object': ['An error occurred %j', { msg: 'This is an error', code: 'ERR' }],
  'unknown': ['hello %a', 'test'],
  'no-replace': [1, 2]
 };

diff --git a/benchmark/util/inspect-proxy.js b/benchmark/util/inspect-proxy.js
@@ -10,20 +10,20 @@ const bench = common.createBenchmark(main, {
 
 function twoDifferentProxies(n) {
  // This one should be slower because we're looking up multiple proxies.
- const proxyA = new Proxy({}, {get: () => {}});
- const proxyB = new Proxy({}, {get: () => {}});
+ const proxyA = new Proxy({}, { get: () => {} });
+ const proxyB = new Proxy({}, { get: () => {} });
  bench.start();
  for (var i = 0; i < n; i += 1)
- util.inspect({a: proxyA, b: proxyB}, {showProxy: true});
+ util.inspect({ a: proxyA, b: proxyB }, { showProxy: true });
  bench.end(n);
 }
 
 function oneProxy(n) {
  // This one should be a bit faster because of the internal caching.
- const proxy = new Proxy({}, {get: () => {}});
+ const proxy = new Proxy({}, { get: () => {} });
  bench.start();
  for (var i = 0; i < n; i += 1)
- util.inspect({a: proxy, b: proxy}, {showProxy: true});
+ util.inspect({ a: proxy, b: proxy }, { showProxy: true });
  bench.end(n);
 }
 

diff --git a/benchmark/util/inspect.js b/benchmark/util/inspect.js
@@ -3,14 +3,14 @@ var util = require('util');
 
 var common = require('../common.js');
 
-var bench = common.createBenchmark(main, {n: [5e6]});
+var bench = common.createBenchmark(main, { n: [5e6] });
 
 function main(conf) {
  var n = conf.n | 0;
 
  bench.start();
  for (var i = 0; i < n; i += 1) {
- util.inspect({a: 'a', b: 'b', c: 'c', d: 'd'});
+ util.inspect({ a: 'a', b: 'b', c: 'c', d: 'd' });
  }
  bench.end(n);
 }
diff --git a/benchmark/vm/run-in-context.js b/benchmark/vm/run-in-context.js
@@ -12,7 +12,7 @@ const vm = require('vm');
 
 function main(conf) {
  const n = +conf.n;
- const options = conf.breakOnSigint ? {breakOnSigint: true} : {};
+ const options = conf.breakOnSigint ? { breakOnSigint: true } : {};
  const withSigintListener = !!conf.withSigintListener;
 
  process.removeAllListeners('SIGINT');

diff --git a/benchmark/vm/run-in-this-context.js b/benchmark/vm/run-in-this-context.js
@@ -12,7 +12,7 @@ const vm = require('vm');
 
 function main(conf) {
  const n = +conf.n;
- const options = conf.breakOnSigint ? {breakOnSigint: true} : {};
+ const options = conf.breakOnSigint ? { breakOnSigint: true } : {};
  const withSigintListener = !!conf.withSigintListener;
 
  process.removeAllListeners('SIGINT');

diff --git a/lib/console.js b/lib/console.js
@@ -146,7 +146,7 @@ Console.prototype.error = Console.prototype.warn;
 
 
 Console.prototype.dir = function dir(object, options) {
- options = Object.assign({customInspect: false}, options);
+ options = Object.assign({ customInspect: false }, options);
  write(this._ignoreErrors,
  this._stdout,
  util.inspect(object, options),

diff --git a/lib/dgram.js b/lib/dgram.js
@@ -411,7 +411,7 @@ Socket.prototype.send = function(buffer,
  this._healthCheck();
 
  if (this._bindState === BIND_STATE_UNBOUND)
- this.bind({port: 0, exclusive: true}, null);
+ this.bind({ port: 0, exclusive: true }, null);
 
  if (list.length === 0)
  list.push(Buffer.alloc(0));

diff --git a/lib/dns.js b/lib/dns.js
@@ -169,7 +169,7 @@ function lookup(hostname, options, callback) {
  if (matchedFamily) {
  if (all) {
  process.nextTick(
- callback, null, [{address: hostname, family: matchedFamily}]);
+ callback, null, [{ address: hostname, family: matchedFamily }]);
  } else {
  process.nextTick(callback, null, hostname, matchedFamily);
  }

diff --git a/lib/fs.js b/lib/fs.js
@@ -254,10 +254,10 @@ function statsFromValues() {
 
 // Don't allow mode to accidentally be overwritten.
 Object.defineProperties(fs, {
- F_OK: {enumerable: true, value: constants.F_OK || 0},
- R_OK: {enumerable: true, value: constants.R_OK || 0},
- W_OK: {enumerable: true, value: constants.W_OK || 0},
- X_OK: {enumerable: true, value: constants.X_OK || 0},
+ F_OK: { enumerable: true, value: constants.F_OK || 0 },
+ R_OK: { enumerable: true, value: constants.R_OK || 0 },
+ W_OK: { enumerable: true, value: constants.W_OK || 0 },
+ X_OK: { enumerable: true, value: constants.X_OK || 0 },
 });
 
 function handleError(val, callback) {

diff --git a/lib/inspector.js b/lib/inspector.js
@@ -66,7 +66,7 @@ class Session extends EventEmitter {
  throw new Error('Session is not connected');
  }
  const id = this[nextIdSymbol]++;
- const message = {id, method};
+ const message = { id, method };
  if (params) {
  message['params'] = params;
  }

diff --git a/lib/internal/bootstrap_node.js b/lib/internal/bootstrap_node.js
@@ -476,7 +476,7 @@
  // wrap it
  source = Module.wrap(source);
  // compile the script, this will throw if it fails
- new vm.Script(source, {displayErrors: true, filename});
+ new vm.Script(source, { displayErrors: true, filename });
  }
 
  // Below you find a minimal module system, which is used to load the node

diff --git a/lib/internal/child_process.js b/lib/internal/child_process.js
@@ -109,7 +109,7 @@ const handleConversion = {
  },
 
  got: function(message, handle, emit) {
- var socket = new net.Socket({handle: handle});
+ var socket = new net.Socket({ handle: handle });
  socket.readable = socket.writable = true;
 
  // if the socket was created by net.Server we will track the socket
@@ -582,7 +582,7 @@ function setupChannel(target, channel) {
  throw new errors.TypeError('ERR_INVALID_ARG_TYPE', 'options', 'Object');
  }
 
- options = Object.assign({swallowErrors: false}, options);
+ options = Object.assign({ swallowErrors: false }, options);
 
  if (this.connected) {
  return this._send(message, handle, options, callback);
@@ -604,7 +604,7 @@ function setupChannel(target, channel) {
 
  // Support legacy function signature
  if (typeof options === 'boolean') {
- options = {swallowErrors: options};
+ options = { swallowErrors: options };
  }
 
  // package messages with a handle object
@@ -838,7 +838,7 @@ function _validateStdio(stdio, sync) {
  }
 
  if (stdio === 'ignore') {
- acc.push({type: 'ignore'});
+ acc.push({ type: 'ignore' });
  } else if (stdio === 'pipe' || typeof stdio === 'number' && stdio < 0) {
  var a = {
  type: 'pipe',

diff --git a/lib/internal/cluster/master.js b/lib/internal/cluster/master.js
@@ -12,7 +12,7 @@ const cluster = new EventEmitter();
 const intercom = new EventEmitter();
 const SCHED_NONE = 1;
 const SCHED_RR = 2;
-const {isLegalPort} = require('internal/net');
+const { isLegalPort } = require('internal/net');
 
 module.exports = cluster;
 

diff --git a/lib/internal/socket_list.js b/lib/internal/socket_list.js
@@ -102,4 +102,4 @@ class SocketListReceive extends EventEmitter {
  }
 }
 
-module.exports = {SocketListSend, SocketListReceive};
+module.exports = { SocketListSend, SocketListReceive };
diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -150,8 +150,8 @@ function createClassWrapper(type) {
  }
  // Mask the wrapper function name and length values
  Object.defineProperties(fn, {
- name: {value: type.name},
- length: {value: type.length}
+ name: { value: type.name },
+ length: { value: type.length }
  });
  Object.setPrototypeOf(fn, type);
  fn.prototype = type.prototype;

diff --git a/lib/readline.js b/lib/readline.js
@@ -683,7 +683,7 @@ Interface.prototype._getDisplayPos = function(str) {
  }
  var cols = offset % col;
  var rows = row + (offset - cols) / col;
- return {cols: cols, rows: rows};
+ return { cols: cols, rows: rows };
 };
 
 
@@ -702,7 +702,7 @@ Interface.prototype._getCursorPos = function() {
  rows++;
  cols = 0;
  }
- return {cols: cols, rows: rows};
+ return { cols: cols, rows: rows };
 };
 
 

diff --git a/lib/repl.js b/lib/repl.js
@@ -1069,7 +1069,7 @@ REPLServer.prototype.parseREPLKeyword = function(keyword, rest) {
 
 REPLServer.prototype.defineCommand = function(keyword, cmd) {
  if (typeof cmd === 'function') {
- cmd = {action: cmd};
+ cmd = { action: cmd };
  } else if (typeof cmd.action !== 'function') {
  throw new Error('Bad argument, "action" command must be a function');
  }

diff --git a/test/addons-napi/test_make_callback_recurse/test.js b/test/addons-napi/test_make_callback_recurse/test.js
@@ -101,11 +101,11 @@ function checkDomains() {
  const d2 = domain.create();
  const d3 = domain.create();
 
- makeCallback({domain: d1}, common.mustCall(function() {
+ makeCallback({ domain: d1 }, common.mustCall(function() {
  assert.strictEqual(d1, process.domain);
- makeCallback({domain: d2}, common.mustCall(function() {
+ makeCallback({ domain: d2 }, common.mustCall(function() {
  assert.strictEqual(d2, process.domain);
- makeCallback({domain: d3}, common.mustCall(function() {
+ makeCallback({ domain: d3 }, common.mustCall(function() {
  assert.strictEqual(d3, process.domain);
  }));
  assert.strictEqual(d2, process.domain);
@@ -119,11 +119,11 @@ function checkDomains() {
  const d2 = domain.create();
  const d3 = domain.create();
 
- makeCallback({domain: d1}, common.mustCall(function() {
+ makeCallback({ domain: d1 }, common.mustCall(function() {
  assert.strictEqual(d1, process.domain);
- makeCallback({domain: d2}, common.mustCall(function() {
+ makeCallback({ domain: d2 }, common.mustCall(function() {
  assert.strictEqual(d2, process.domain);
- makeCallback({domain: d3}, common.mustCall(function() {
+ makeCallback({ domain: d3 }, common.mustCall(function() {
  assert.strictEqual(d3, process.domain);
  }));
  assert.strictEqual(d2, process.domain);
@@ -139,7 +139,7 @@ function checkDomains() {
  d.on('error', common.mustCall(function(e) {
  assert.strictEqual(e.message, `throw from domain ${id}`);
  }));
- makeCallback({domain: d}, function() {
+ makeCallback({ domain: d }, function() {
  throw new Error(`throw from domain ${id}`);
  });
  throw new Error('UNREACHABLE');

diff --git a/test/addons-napi/test_object/test.js b/test/addons-napi/test_object/test.js
@@ -92,12 +92,12 @@ assert.strictEqual(newObject.test_string, 'test string');
  z: 10
  };
 
- assert.deepStrictEqual(test_object.Inflate(cube), {x: 11, y: 11, z: 11});
- assert.deepStrictEqual(test_object.Inflate(cube), {x: 12, y: 12, z: 12});
- assert.deepStrictEqual(test_object.Inflate(cube), {x: 13, y: 13, z: 13});
+ assert.deepStrictEqual(test_object.Inflate(cube), { x: 11, y: 11, z: 11 });
+ assert.deepStrictEqual(test_object.Inflate(cube), { x: 12, y: 12, z: 12 });
+ assert.deepStrictEqual(test_object.Inflate(cube), { x: 13, y: 13, z: 13 });
  cube.t = 13;
  assert.deepStrictEqual(
- test_object.Inflate(cube), {x: 14, y: 14, z: 14, t: 14});
+ test_object.Inflate(cube), { x: 14, y: 14, z: 14, t: 14 });
 
  const sym1 = Symbol('1');
  const sym2 = Symbol('2');