From 9e7fd8e810dc5f2365201e09ad3eaa6d34c52e14 Mon Sep 17 00:00:00 2001 From: Adrian Nitu Date: Thu, 4 Aug 2016 12:00:59 +0300 Subject: [PATCH] benchmark: fix comment typos and code format I noticed some typos and the lack of {} following an if. PR-URL: https://github.com/nodejs/node/pull/7961 Reviewed-By: James M Snell Reviewed-By: Andreas Madsen --- benchmark/common.js | 10 +++++----- benchmark/compare.js | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/benchmark/common.js b/benchmark/common.js index 669a4c642b2bfd..3807fea7957096 100644 --- a/benchmark/common.js +++ b/benchmark/common.js @@ -30,7 +30,7 @@ function Benchmark(fn, options) { Benchmark.prototype._parseArgs = function(argv, options) { const cliOptions = Object.assign({}, options); - // Parse configuarion arguments + // Parse configuration arguments for (const arg of argv) { const match = arg.match(/^(.+?)=([\s\S]*)$/); if (!match || !match[1]) { @@ -52,7 +52,7 @@ Benchmark.prototype._queue = function(options) { const queue = []; const keys = Object.keys(options); - // Perform a depth-first walk though all options to genereate a + // Perform a depth-first walk though all options to generate a // configuration list that contains all combinations. function recursive(keyIndex, prevConfig) { const key = keys[keyIndex]; @@ -171,9 +171,9 @@ Benchmark.prototype._run = function() { }; Benchmark.prototype.start = function() { - if (this._started) + if (this._started) { throw new Error('Called start more than once in a single benchmark'); - + } this._started = true; this._time = process.hrtime(); }; @@ -195,7 +195,7 @@ Benchmark.prototype.end = function(operations) { }; function formatResult(data) { - // Construct confiuration string, " A=a, B=b, ..." + // Construct configuration string, " A=a, B=b, ..." let conf = ''; for (const key of Object.keys(data.conf)) { conf += ' ' + key + '=' + JSON.stringify(data.conf[key]); diff --git a/benchmark/compare.js b/benchmark/compare.js index fb179e0e4703ed..94ff19bb56701a 100644 --- a/benchmark/compare.js +++ b/benchmark/compare.js @@ -65,7 +65,7 @@ console.log('"binary", "filename", "configuration", "rate", "time"'); } conf = conf.slice(1); - // Escape qoutes (") for correct csv formatting + // Escape quotes (") for correct csv formatting conf = conf.replace(/"/g, '""'); console.log(`"${job.binary}", "${job.filename}", "${conf}", ` +