benchmark: (tls) use destructuring

BridgeAR · BridgeAR · commit e49d86437d60 · 2018-01-23T01:29:18.000+01:00
PR-URL: nodejs#18250 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/benchmark/tls/convertprotocols.js b/benchmark/tls/convertprotocols.js
@@ -7,9 +7,7 @@ const bench = common.createBenchmark(main, {
   n: [1, 50000]
 });
 
-function main(conf) {
-  const n = +conf.n;
-
+function main({ n }) {
   var i = 0;
   var m = {};
   // First call dominates results
diff --git a/benchmark/tls/throughput.js b/benchmark/tls/throughput.js
@@ -6,20 +6,15 @@ const bench = common.createBenchmark(main, {
   size: [2, 1024, 1024 * 1024]
 });
 
-var dur, type, encoding, size;
-var server;
-
 const path = require('path');
 const fs = require('fs');
 const cert_dir = path.resolve(__dirname, '../../test/fixtures');
 var options;
 const tls = require('tls');
 
-function main(conf) {
-  dur = +conf.dur;
-  type = conf.type;
-  size = +conf.size;
-
+function main({ dur, type, size }) {
+  var encoding;
+  var server;
   var chunk;
   switch (type) {
     case 'buf':
diff --git a/benchmark/tls/tls-connect.js b/benchmark/tls/tls-connect.js
@@ -16,10 +16,7 @@ var dur;
 var concurrency;
 var running = true;
 
-function main(conf) {
-  dur = +conf.dur;
-  concurrency = +conf.concurrency;
-
+function main({ dur, concurrency }) {
   const cert_dir = path.resolve(__dirname, '../../test/fixtures');
   const options = {
     key: fs.readFileSync(`${cert_dir}/test_key.pem`),
