lib,benchmark,test: implement consistent braces

This change is in preparation for lint-enforced brace style.

PR-URL: https://github.com/nodejs/node/pull/7630
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>

benchmark/child_process/child-process-read-ipc.js

@@ -1,6 +1,5 @@
 'use strict';
-if (process.argv[2] === 'child')
+if (process.argv[2] === 'child') {
-{
   const len = +process.argv[3];
   const msg = `"${'.'.repeat(len)}"`;
   while (true) {

lib/_stream_readable.js

@@ -803,9 +803,11 @@ Readable.prototype.wrap = function(stream) {
   // important when wrapping filters and duplexes.
   for (var i in stream) {
     if (this[i] === undefined && typeof stream[i] === 'function') {
-      this[i] = function(method) { return function() {
+      this[i] = function(method) {
-        return stream[method].apply(stream, arguments);
+        return function() {
-      }; }(i);
+          return stream[method].apply(stream, arguments);
+        };
+      }(i);
     }
   }
 

lib/module.js

@@ -622,8 +622,7 @@ Module._preloadModules = function(requests) {
   var parent = new Module('internal/preload', null);
   try {
     parent.paths = Module._nodeModulePaths(process.cwd());
-  }
+  } catch (e) {
-  catch (e) {
     if (e.code !== 'ENOENT') {
       throw e;
     }

lib/net.js

@@ -1159,8 +1159,7 @@ function createServerHandle(address, port, addressType, fd) {
   if (typeof fd === 'number' && fd >= 0) {
     try {
       handle = createHandle(fd);
-    }
+    } catch (e) {
-    catch (e) {
       // Not a fd we can listen on.  This will trigger an error.
       debug('listen invalid fd=' + fd + ': ' + e.message);
       return uv.UV_EINVAL;

test/internet/test-dgram-broadcast-multi-process.js

@@ -96,8 +96,7 @@ if (process.argv[2] !== 'child') {
             //all child process are listening, so start sending
             sendSocket.sendNext();
           }
-        }
+        } else if (msg.message) {
-        else if (msg.message) {
           worker.messagesReceived.push(msg.message);
 
           if (worker.messagesReceived.length === messages.length) {

test/parallel/test-child-process-fork-exec-path.js

@@ -12,13 +12,11 @@ if (process.env.FORK) {
   assert.equal(process.argv[0], copyPath);
   process.send(msg);
   process.exit();
-}
+} else {
-else {
   common.refreshTmpDir();
   try {
     fs.unlinkSync(copyPath);
-  }
+  } catch (e) {
-  catch (e) {
     if (e.code !== 'ENOENT') throw e;
   }
   fs.writeFileSync(copyPath, fs.readFileSync(nodePath));

test/parallel/test-child-process-recv-handle.js

@@ -49,14 +49,12 @@ function worker() {
     if (n === 1) {
       assert.equal(msg, 'one');
       assert.equal(handle, undefined);
-    }
+    } else if (n === 2) {
-    else if (n === 2) {
       assert.equal(msg, 'two');
       assert.equal(typeof handle, 'object');  // Also matches null, therefore...
       assert.ok(handle);                      // also check that it's truthy.
       handle.close();
-    }
+    } else if (n === 3) {
-    else if (n === 3) {
       assert.equal(msg, 'three');
       assert.equal(handle, undefined);
       process.exit();

test/parallel/test-cluster-basic.js

@@ -18,9 +18,7 @@ if (cluster.isWorker) {
   http.Server(function() {
 
   }).listen(common.PORT, '127.0.0.1');
-}
+} else if (cluster.isMaster) {
-
-else if (cluster.isMaster) {
 
   var checks = {
     cluster: {

test/parallel/test-cluster-bind-privileged-port.js

@@ -18,8 +18,7 @@ if (cluster.isMaster) {
   cluster.fork().on('exit', common.mustCall(function(exitCode) {
     assert.equal(exitCode, 0);
   }));
-}
+} else {
-else {
   var s = net.createServer(common.fail);
   s.listen(42, common.fail.bind(null, 'listen should have failed'));
   s.on('error', common.mustCall(function(err) {

test/parallel/test-cluster-bind-twice.js

@@ -64,8 +64,7 @@ if (!id) {
   process.on('exit', function() {
     assert(ok);
   });
-}
+} else if (id === 'one') {
-else if (id === 'one') {
   if (cluster.isMaster) return startWorker();
 
   http.createServer(common.fail).listen(common.PORT, function() {
@@ -75,8 +74,7 @@ else if (id === 'one') {
   process.on('message', function(m) {
     if (m === 'QUIT') process.exit();
   });
-}
+} else if (id === 'two') {
-else if (id === 'two') {
   if (cluster.isMaster) return startWorker();
 
   let ok = false;
@@ -96,8 +94,7 @@ else if (id === 'two') {
       ok = true;
     });
   });
-}
+} else {
-else {
   assert(0); // bad command line argument
 }
 

test/parallel/test-cluster-eaddrinuse.js

@@ -21,8 +21,7 @@ if (id === 'undefined') {
       });
     });
   });
-}
+} else if (id === 'worker') {
-else if (id === 'worker') {
   let server = net.createServer(common.fail);
   server.listen(common.PORT, common.fail);
   server.on('error', common.mustCall(function(e) {
@@ -36,7 +35,6 @@ else if (id === 'worker') {
       }));
     });
   }));
-}
+} else {
-else {
   assert(0);  // Bad argument.
 }

test/parallel/test-cluster-listening-port.js

@@ -19,7 +19,6 @@ if (cluster.isMaster) {
     // ensure that the 'listening' handler has been called
     assert(port);
   });
-}
+} else {
-else {
   net.createServer(common.fail).listen(0);
 }

test/parallel/test-cluster-message.js

@@ -40,9 +40,7 @@ if (cluster.isWorker) {
   });
 
   server.listen(common.PORT, '127.0.0.1');
-}
+} else if (cluster.isMaster) {
-
-else if (cluster.isMaster) {
 
   var checks = {
     global: {

test/parallel/test-cluster-net-listen.js

@@ -14,8 +14,7 @@ if (cluster.isMaster) {
   process.on('exit', function() {
     assert.equal(worker, null);
   });
-}
+} else {
-else {
   // listen() without port should not trigger a libuv assert
   net.createServer(common.fail).listen(process.exit);
 }

test/parallel/test-cluster-shared-handle-bind-error.js

@@ -16,8 +16,7 @@ if (cluster.isMaster) {
       server.close();
     }));
   });
-}
+} else {
-else {
   var s = net.createServer(common.fail);
   s.listen(common.PORT, common.fail.bind(null, 'listen should have failed'));
   s.on('error', common.mustCall(function(err) {

test/parallel/test-cluster-shared-handle-bind-privileged-port.js

@@ -20,8 +20,7 @@ if (cluster.isMaster) {
   cluster.fork().on('exit', common.mustCall(function(exitCode) {
     assert.equal(exitCode, 0);
   }));
-}
+} else {
-else {
   var s = net.createServer(common.fail);
   s.listen(42, common.fail.bind(null, 'listen should have failed'));
   s.on('error', common.mustCall(function(err) {

test/parallel/test-cluster-uncaught-exception.js

@@ -23,8 +23,7 @@ if (isTestRunner) {
   master.on('exit', function(code) {
     exitCode = code;
   });
-}
+} else if (cluster.isMaster) {
-else if (cluster.isMaster) {
   process.on('uncaughtException', function() {
     process.nextTick(function() {
       process.exit(MAGIC_EXIT_CODE);
@@ -33,7 +32,6 @@ else if (cluster.isMaster) {
 
   cluster.fork();
   throw new Error('kill master');
-}
+} else { // worker
-else { // worker
   process.exit();
 }

test/parallel/test-cluster-worker-death.js

@@ -5,8 +5,7 @@ var cluster = require('cluster');
 
 if (!cluster.isMaster) {
   process.exit(42);
-}
+} else {
-else {
   var seenExit = 0;
   var seenDeath = 0;
   var worker = cluster.fork();

test/parallel/test-crypto-authenticated.js

@@ -362,8 +362,7 @@ for (var i in TEST_CASES) {
   (function() {
     if (!test.password) return;
     if (common.hasFipsCrypto) {
-      assert.throws(function()
+      assert.throws(() => { crypto.createCipher(test.algo, test.password); },
-                    { crypto.createCipher(test.algo, test.password); },
                     /not supported in FIPS mode/);
     } else {
       var encrypt = crypto.createCipher(test.algo, test.password);
@@ -383,8 +382,7 @@ for (var i in TEST_CASES) {
   (function() {
     if (!test.password) return;
     if (common.hasFipsCrypto) {
-      assert.throws(function()
+      assert.throws(() => { crypto.createDecipher(test.algo, test.password); },
-                    { crypto.createDecipher(test.algo, test.password); },
                     /not supported in FIPS mode/);
     } else {
       var decrypt = crypto.createDecipher(test.algo, test.password);
@@ -415,9 +413,9 @@ for (var i in TEST_CASES) {
       'ipxp9a6i1Mb4USb4', '6fKjEjR3Vl30EUYC');
     encrypt.update('blah', 'ascii');
     encrypt.final();
-    assert.throws(function() { encrypt.getAuthTag(); }, / state/);
+    assert.throws(() => { encrypt.getAuthTag(); }, / state/);
-    assert.throws(function() {
+    assert.throws(() => { encrypt.setAAD(Buffer.from('123', 'ascii')); },
-      encrypt.setAAD(Buffer.from('123', 'ascii')); }, / state/);
+                  / state/);
   })();
 
   (function() {
@@ -432,8 +430,8 @@ for (var i in TEST_CASES) {
     // trying to set tag on encryption object:
     var encrypt = crypto.createCipheriv(test.algo,
       Buffer.from(test.key, 'hex'), Buffer.from(test.iv, 'hex'));
-    assert.throws(function() {
+    assert.throws(() => { encrypt.setAuthTag(Buffer.from(test.tag, 'hex')); },
-      encrypt.setAuthTag(Buffer.from(test.tag, 'hex')); }, / state/);
+                  / state/);
   })();
 
   (function() {

test/parallel/test-debugger-util-regression.js

@@ -39,12 +39,10 @@ proc.stdout.on('data', (data) => {
       stdout.includes('> 4') && nextCount < 4) {
     nextCount++;
     proc.stdin.write('n\n');
-  }
+  } else if (stdout.includes('{ a: \'b\' }')) {
-  else if (stdout.includes('{ a: \'b\' }')) {
     clearTimeout(timer);
     proc.stdin.write('.exit\n');
-  }
+  } else if (stdout.includes('program terminated')) {
-  else if (stdout.includes('program terminated')) {
     // Catch edge case present in v4.x
     // process will terminate after call to util.inspect
     common.fail('the program should not terminate');

test/parallel/test-fs-open.js

@@ -8,8 +8,7 @@ try {
   // should throw ENOENT, not EBADF
   // see https://github.com/joyent/node/pull/1228
   fs.openSync('/path/to/file/that/does/not/exist', 'r');
-}
+} catch (e) {
-catch (e) {
   assert.equal(e.code, 'ENOENT');
   caughtException = true;
 }

test/parallel/test-fs-realpath.js

@@ -57,8 +57,7 @@ function asynctest(testBlock, args, callback, assertBlock) {
     if (assertBlock) {
       try {
         ignoreError = assertBlock.apply(assertBlock, arguments);
-      }
+      } catch (e) {
-      catch (e) {
         err = e;
       }
     }

test/parallel/test-http-server-stale-close.js

@@ -13,8 +13,7 @@ if (process.env.NODE_TEST_FORK_PORT) {
   }, process.exit);
   req.write('BAM');
   req.end();
-}
+} else {
-else {
   var server = http.createServer(function(req, res) {
     res.writeHead(200, {'Content-Length': '42'});
     req.pipe(res);

test/parallel/test-next-tick-errors.js

@@ -40,8 +40,7 @@ process.on('uncaughtException', function() {
   if (!exceptionHandled) {
     exceptionHandled = true;
     order.push('B');
-  }
+  } else {
-  else {
     // If we get here then the first process.nextTick got called twice
     order.push('OOPS!');
   }

test/parallel/test-process-argv-0.js

@@ -16,7 +16,6 @@ if (process.argv[2] !== 'child') {
   process.on('exit', function() {
     assert.equal(childArgv0, process.execPath);
   });
-}
+} else {
-else {
   process.stdout.write(process.argv[0]);
 }

test/parallel/test-process-cpuUsage.js

@@ -42,14 +42,18 @@ assert.throws(function() { process.cpuUsage({ user: null, system: 'c' }); });
 assert.throws(function() { process.cpuUsage({ user: 'd', system: null }); });
 assert.throws(function() { process.cpuUsage({ user: -1, system: 2 }); });
 assert.throws(function() { process.cpuUsage({ user: 3, system: -2 }); });
-assert.throws(function() { process.cpuUsage({
+assert.throws(function() {
-  user: Number.POSITIVE_INFINITY,
+  process.cpuUsage({
-  system: 4
+    user: Number.POSITIVE_INFINITY,
-});});
+    system: 4
-assert.throws(function() { process.cpuUsage({
+  });
-  user: 5,
+});
-  system: Number.NEGATIVE_INFINITY
+assert.throws(function() {
-});});
+  process.cpuUsage({
+    user: 5,
+    system: Number.NEGATIVE_INFINITY
+  });
+});
 
 // Ensure that the return value is the expected shape.
 function validateResult(result) {

test/parallel/test-repl-domain.js

@@ -11,8 +11,7 @@ putIn.write = function(data) {
   // give a false negative.  Don't throw, just print and exit.
   if (data === 'OK\n') {
     console.log('ok');
-  }
+  } else {
-  else {
     console.error(data);
     process.exit(1);
   }

test/parallel/test-vm-context.js

@@ -32,8 +32,7 @@ console.error('test runInContext signature');
 var gh1140Exception;
 try {
   vm.runInContext('throw new Error()', context, 'expected-filename.js');
-}
+} catch (e) {
-catch (e) {
   gh1140Exception = e;
   assert.ok(/expected-filename/.test(e.stack),
             'expected appearance of filename in Error stack');

test/pummel/test-fs-watch-file-slow.js

@@ -11,8 +11,7 @@ var nevents = 0;
 
 try {
   fs.unlinkSync(FILENAME);
-}
+} catch (e) {
-catch (e) {
   // swallow
 }
 

test/pummel/test-regress-GH-814.js

@@ -44,15 +44,13 @@ var timeToQuit = Date.now() + 8e3; //Test during no more than this seconds.
     if (bufPool.push(nuBuf) > 100) {
       bufPool.length = 0;
     }
-  }
+  } else {
-  else {
     throw new Error("Buffer GC'ed test -> FAIL");
   }
 
   if (Date.now() < timeToQuit) {
     process.nextTick(main);
-  }
+  } else {
-  else {
     tail.kill();
     console.log("Buffer GC'ed test -> PASS (OK)");
   }