lib,src: fix consistent spacing inside braces

PR-URL: nodejs#14162 Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Timothy Gu <[email protected]> Reviewed-By: James M Snell <[email protected]>
refack · Jul 21, 2017 · bb29405 · bb29405
1 parent 4f87522
commit bb29405
Show file tree

Hide file tree

Showing 293 changed files with 1,108 additions and 1,051 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
@@ -124,7 +124,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)}\n`,

diff --git a/lib/dgram.js b/lib/dgram.js
@@ -408,7 +408,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
@@ -164,7 +164,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
@@ -253,10 +253,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
@@ -62,7 +62,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
@@ -454,7 +454,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
@@ -110,7 +110,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
@@ -584,7 +584,7 @@ function setupChannel(target, channel) {
                                  options);
     }
 
-    options = Object.assign({swallowErrors: false}, options);
+    options = Object.assign({ swallowErrors: false }, options);
 
     if (this.connected) {
       return this._send(message, handle, options, callback);
@@ -606,7 +606,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
@@ -840,7 +840,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
@@ -104,4 +104,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
@@ -692,7 +692,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 };
 };
 
 
@@ -711,7 +711,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
@@ -1055,7 +1055,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 errors.TypeError('ERR_INVALID_ARG_TYPE',
       'action', 'function', cmd.action);

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');