Skip to content

Commit

Permalink
benchmark: fix linting issues
Browse files Browse the repository at this point in the history
PR-URL: #5773
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
  • Loading branch information
Trott authored and Myles Borins committed Mar 21, 2016
1 parent b983765 commit 1a829af
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion benchmark/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ function parseOpts(options) {
if (!match || !match[1] || !match[2] || !options[match[1]]) {
return null;
} else {
conf[match[1]] = isFinite(match[2]) ? +match[2] : match[2]
conf[match[1]] = isFinite(match[2]) ? +match[2] : match[2];
num--;
}
}
Expand Down
4 changes: 2 additions & 2 deletions benchmark/misc/domain-fn-args.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,15 +12,15 @@ var gargs = [1, 2, 3];

function main(conf) {

var args, ret, n = +conf.n;
var args, n = +conf.n;
var myArguments = gargs.slice(0, conf.arguments);
bench.start();

bdomain.enter();
for (var i = 0; i < n; i++) {
if (myArguments.length >= 2) {
args = Array.prototype.slice.call(myArguments, 1);
ret = fn.apply(this, args);
fn.apply(this, args);
} else {
fn.call(this);
}
Expand Down
4 changes: 2 additions & 2 deletions benchmark/misc/url.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
'use strict';
var url = require('url')
var url = require('url');
var n = 25 * 100;

var urls = [
Expand All @@ -21,7 +21,7 @@ benchmark('parse()', url.parse);
benchmark('format()', url.format);
paths.forEach(function(p) {
benchmark('resolve("' + p + '")', function(u) {
url.resolve(u, p)
url.resolve(u, p);
});
});

Expand Down
1 change: 1 addition & 0 deletions benchmark/net/dgram.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
'use strict';
// test UDP send/recv throughput

var common = require('../common.js');
Expand Down
1 change: 1 addition & 0 deletions benchmark/path/relative.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
'use strict';
var common = require('../common.js');
var path = require('path');
var v8 = require('v8');
Expand Down

0 comments on commit 1a829af

Please sign in to comment.