benchmark: fix lint errors

PR-URL: https://github.com/nodejs/node/pull/5773
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>

diff --git a/benchmark/arrays/var-int.js b/benchmark/arrays/var-int.js
index 47a7e62dc7681bf3812d7df9f02a3646ae9fa4c4..2b5fefc126ac1f6f8372efba1f3fb242bbf611d6 100644 (file)
--- a/benchmark/arrays/var-int.js
+++ b/benchmark/arrays/var-int.js
@@ -1,6 +1,8 @@
 var common = require('../common.js');
 var bench = common.createBenchmark(main, {
-  type: 'Array Buffer Int8Array Uint8Array Int16Array Uint16Array Int32Array Uint32Array Float32Array Float64Array'.split(' '),
+  type: ['Array', 'Buffer', 'Int8Array', 'Uint8Array', 'Int16Array',
+         'Uint16Array', 'Int32Array', 'Uint32Array', 'Float32Array',
+         'Float64Array'],
   n: [25]
 });
 

diff --git a/benchmark/arrays/zero-float.js b/benchmark/arrays/zero-float.js
index a6624205bffdea16f9cb50369702d90d1bf41995..015e9a08f95725c70ae398ae1e0e5dbf1e46bbb4 100644 (file)
--- a/benchmark/arrays/zero-float.js
+++ b/benchmark/arrays/zero-float.js
@@ -1,6 +1,8 @@
 var common = require('../common.js');
 var bench = common.createBenchmark(main, {
-  type: 'Array Buffer Int8Array Uint8Array Int16Array Uint16Array Int32Array Uint32Array Float32Array Float64Array'.split(' '),
+  type: ['Array', 'Buffer', 'Int8Array', 'Uint8Array', 'Int16Array',
+         'Uint16Array', 'Int32Array', 'Uint32Array', 'Float32Array',
+         'Float64Array'],
   n: [25]
 });
 

diff --git a/benchmark/arrays/zero-int.js b/benchmark/arrays/zero-int.js
index 29a2d58b665cbe6275a2a4d6dea9a6cce4f25bdc..4dad3e9980e78b899accf992b5a97f8386da6570 100644 (file)
--- a/benchmark/arrays/zero-int.js
+++ b/benchmark/arrays/zero-int.js
@@ -1,6 +1,8 @@
 var common = require('../common.js');
 var bench = common.createBenchmark(main, {
-  type: 'Array Buffer Int8Array Uint8Array Int16Array Uint16Array Int32Array Uint32Array Float32Array Float64Array'.split(' '),
+  type: ['Array', 'Buffer', 'Int8Array', 'Uint8Array', 'Int16Array',
+         'Uint16Array', 'Int32Array', 'Uint32Array', 'Float32Array',
+         'Float64Array'],
   n: [25]
 });
 

diff --git a/benchmark/buffers/buffer-creation.js b/benchmark/buffers/buffer-creation.js
index bc0c55711858de3cda375b3e08bb163d272b57e2..fc498d0bf717d76fec52e0349ab106ae027f482b 100644 (file)
--- a/benchmark/buffers/buffer-creation.js
+++ b/benchmark/buffers/buffer-creation.js
@@ -13,7 +13,7 @@ function main(conf) {
   var clazz = conf.type === 'fast' ? Buffer : SlowBuffer;
   bench.start();
   for (var i = 0; i < n * 1024; i++) {
-    b = new clazz(len);
+    new clazz(len);
   }
   bench.end(n);
 }

diff --git a/benchmark/buffers/buffer-indexof.js b/benchmark/buffers/buffer-indexof.js
index c575f54b6a638c963dd729b0ba3b2ec28166c28d..0f54930296f2685d3e180a25764c127953a163e4 100644 (file)
--- a/benchmark/buffers/buffer-indexof.js
+++ b/benchmark/buffers/buffer-indexof.js
@@ -1,5 +1,6 @@
 var common = require('../common.js');
 var fs = require('fs');
+const path = require('path');
 
 var bench = common.createBenchmark(main, {
   search: ['@', 'SQ', '10x', '--l', 'Alice', 'Gryphon', 'Panther',
@@ -14,7 +15,9 @@ var bench = common.createBenchmark(main, {
 
 function main(conf) {
   var iter = (conf.iter) * 100000;
-  var aliceBuffer = fs.readFileSync(__dirname + '/../fixtures/alice.html');
+  var aliceBuffer = fs.readFileSync(
+    path.resolve(__dirname, '../fixtures/alice.html')
+  );
   var search = conf.search;
   var encoding = conf.encoding;
 

diff --git a/benchmark/compare.js b/benchmark/compare.js
index edb116d0c64baab467aecf61a8d570fea15f94b6..94d8cb515fbb03d6027e0956a27563ecffa5bdb6 100644 (file)
--- a/benchmark/compare.js
+++ b/benchmark/compare.js
@@ -75,10 +75,15 @@ function run() {
 
   var out = '';
   var child;
-  if (Array.isArray(benchmarks) && benchmarks.length)
-    child = spawn(node, ['benchmark/common.js'].concat(benchmarks), { env: env });
-  else
+  if (Array.isArray(benchmarks) && benchmarks.length) {
+    child = spawn(
+      node,
+      ['benchmark/common.js'].concat(benchmarks),
+      { env: env }
+    );
+  } else {
     child = spawn('make', [runBench], { env: env });
+  }
   child.stdout.setEncoding('utf8');
   child.stdout.on('data', function(c) {
     out += c;
@@ -134,9 +139,20 @@ function compare() {
     if (show === 'green' && !g || show === 'red' && !r)
       return;
 
-    var r0 = util.format('%s%s: %d%s', g, nodes[0], n0.toPrecision(5), g ? reset : '');
-    var r1 = util.format('%s%s: %d%s', r, nodes[1], n1.toPrecision(5), r ? reset : '');
-    var pct = c + pct + '%' + reset;
+    var r0 = util.format(
+      '%s%s: %d%s',
+      g,
+      nodes[0],
+      n0.toPrecision(5), g ? reset : ''
+    );
+    var r1 = util.format(
+      '%s%s: %d%s',
+      r,
+      nodes[1],
+      n1.toPrecision(5), r ? reset : ''
+    );
+    pct = c + pct + '%' + reset;
+
     var l = util.format('%s: %s %s', bench, r0, r1);
     maxLen = Math.max(l.length + pct.length, maxLen);
     return [l, pct];

diff --git a/benchmark/http_simple_auto.js b/benchmark/http_simple_auto.js
index f72cc01e60aa46fbe69e8c06e2bf9ffe90f067e1..0769f834081c7ca181eb9e0252fe4f22d5e3fbd1 100644 (file)
--- a/benchmark/http_simple_auto.js
+++ b/benchmark/http_simple_auto.js
@@ -109,10 +109,10 @@ server.listen(port, function () {
 });
 
 function dump_mm_stats() {
-  if (typeof gc != 'function') return;
+  if (typeof global.gc != 'function') return;
 
   var before = process.memoryUsage();
-  for (var i = 0; i < 10; ++i) gc();
+  for (var i = 0; i < 10; ++i) global.gc();
   var after = process.memoryUsage();
   setTimeout(print_stats, 250); // give GC time to settle
 

diff --git a/benchmark/http_simple_cluster.js b/benchmark/http_simple_cluster.js
index 9a886f2ee75dabafb73cadba982fd99c79c68034..f48a2a2599a78f55f43b71cac2b86cffa6e5dd94 100644 (file)
--- a/benchmark/http_simple_cluster.js
+++ b/benchmark/http_simple_cluster.js
@@ -1,9 +1,11 @@
-var cluster = require('cluster');
-var os = require('os');
+'use strict';
+const cluster = require('cluster');
+const os = require('os');
+const path = require('path');
 
 if (cluster.isMaster) {
   console.log('master running on pid %d', process.pid);
   for (var i = 0, n = os.cpus().length; i < n; ++i) cluster.fork();
 } else {
-  require(__dirname + '/http_simple.js');
+  require(path.join(__dirname, 'http_simple.js'));
 }

diff --git a/benchmark/idle_clients.js b/benchmark/idle_clients.js
index da96b8ac654db4cc0773b26d9270db74b1fbcb16..f061ff4d859836e300b8d57ae8ac03cef0a799ec 100644 (file)
--- a/benchmark/idle_clients.js
+++ b/benchmark/idle_clients.js
@@ -18,7 +18,6 @@ function connect () {
 
     s.on('close', function () {
       if (gotConnected) connections--;
-      lastClose = new Date();
     });
 
     s.on('error', function () {

diff --git a/benchmark/misc/domain-fn-args.js b/benchmark/misc/domain-fn-args.js
index 36e9e900299bb24573ec411f354410b19430d2b4..9cc17fa8fd94f5ae30ba7b3a53fbe65f31a5f8db 100644 (file)
--- a/benchmark/misc/domain-fn-args.js
+++ b/benchmark/misc/domain-fn-args.js
@@ -21,7 +21,7 @@ function main(conf) {
       args = Array.prototype.slice.call(arguments, 1);
       ret = fn.apply(this, args);
     } else {
-      ret = fn.call(this);
+      fn.call(this);
     }
   }
   bdomain.exit();

diff --git a/benchmark/misc/freelist.js b/benchmark/misc/freelist.js
index 76275000b08ffb471bb37c52c3237c874792a214..7956dc3d6b18c0d3d316adc423db2bfd1f4e9d99 100644 (file)
--- a/benchmark/misc/freelist.js
+++ b/benchmark/misc/freelist.js
@@ -22,7 +22,7 @@ function main(conf) {
 
   bench.start();
 
-  for (i = 0; i < n; i++){
+  for (i = 0; i < n; i++) {
     // Return all the items to the pool
     for (j = 0; j < poolSize; j++) {
       list.free(used[j]);

diff --git a/benchmark/misc/module-loader.js b/benchmark/misc/module-loader.js
index e53bc4d801164bd935ed4a5652a1d40a564bea22..fcda0a288b9e54a1fe3c491a26c5cb4b3d665008 100644 (file)
--- a/benchmark/misc/module-loader.js
+++ b/benchmark/misc/module-loader.js
@@ -18,8 +18,14 @@ function main(conf) {
   var n = +conf.thousands * 1e3;
   for (var i = 0; i <= n; i++) {
     fs.mkdirSync(benchmarkDirectory + i);
-    fs.writeFileSync(benchmarkDirectory + i + '/package.json', '{"main": "index.js"}');
-    fs.writeFileSync(benchmarkDirectory + i + '/index.js', 'module.exports = "";');
+    fs.writeFileSync(
+      benchmarkDirectory + i + '/package.json',
+      '{"main": "index.js"}'
+    );
+    fs.writeFileSync(
+      benchmarkDirectory + i + '/index.js',
+      'module.exports = "";'
+    );
   }
 
   measure(n);

diff --git a/benchmark/misc/v8-bench.js b/benchmark/misc/v8-bench.js
index 0eb95aa9e2b2bcf23440ae934519201376f8449d..efe81f8ebb8a0eefb3ab4bc61012972ad7860136 100644 (file)
--- a/benchmark/misc/v8-bench.js
+++ b/benchmark/misc/v8-bench.js
@@ -19,4 +19,4 @@ global.load = function(filename) {
   global.RegExp = $RegExp;
 };
 
-load('run.js');
+global.load('run.js');

diff --git a/benchmark/tls/throughput.js b/benchmark/tls/throughput.js
index 442eba338ed643f303ca452615a35c317d2b9c08..e4873e651fcb2576a0c3a556d2f005bdc9d374b4 100644 (file)
--- a/benchmark/tls/throughput.js
+++ b/benchmark/tls/throughput.js
@@ -44,9 +44,10 @@ function main(conf) {
 
   server = tls.createServer(options, onConnection);
   setTimeout(done, dur * 1000);
+  var conn;
   server.listen(common.PORT, function() {
     var opt = { port: common.PORT, rejectUnauthorized: false };
-    var conn = tls.connect(opt, function() {
+    conn = tls.connect(opt, function() {
       bench.start();
       conn.on('drain', write);
       write();
@@ -68,7 +69,8 @@ function main(conf) {
   function done() {
     var mbits = (received * 8) / (1024 * 1024);
     bench.end(mbits);
-    conn.destroy();
+    if (conn)
+      conn.destroy();
     server.close();
   }
 }