diff options
author | Anna Henningsen <anna@addaleax.net> | 2016-08-30 17:16:09 +0200 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2016-09-11 20:52:57 -0700 |
commit | efabc6ae4db9a79ab959296a54b5732532abd339 (patch) | |
tree | 327ec6ed0eec32690081264df0ebc132459fc5f1 /benchmark | |
parent | 3207ea4e36d22d4cc2641809b4893d87d3bcb04a (diff) | |
download | android-node-v8-efabc6ae4db9a79ab959296a54b5732532abd339.tar.gz android-node-v8-efabc6ae4db9a79ab959296a54b5732532abd339.tar.bz2 android-node-v8-efabc6ae4db9a79ab959296a54b5732532abd339.zip |
benchmark: fix off-by-one error in fs benchmarks
Fix a off-by-one error that made the benchmarks for asynchronous
functions run `n - 1` times instead of `n` times.
PR-URL: https://github.com/nodejs/node/pull/8338
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Diffstat (limited to 'benchmark')
-rw-r--r-- | benchmark/fs/bench-readdir.js | 2 | ||||
-rw-r--r-- | benchmark/fs/bench-realpath.js | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/benchmark/fs/bench-readdir.js b/benchmark/fs/bench-readdir.js index 565b92d149..eb15e72724 100644 --- a/benchmark/fs/bench-readdir.js +++ b/benchmark/fs/bench-readdir.js @@ -14,7 +14,7 @@ function main(conf) { bench.start(); (function r(cntr) { - if (--cntr <= 0) + if (cntr-- <= 0) return bench.end(n); fs.readdir(path.resolve(__dirname, '../../lib/'), function() { r(cntr); diff --git a/benchmark/fs/bench-realpath.js b/benchmark/fs/bench-realpath.js index 1a181935f1..384276eaff 100644 --- a/benchmark/fs/bench-realpath.js +++ b/benchmark/fs/bench-realpath.js @@ -27,7 +27,7 @@ function main(conf) { function relativePath(n) { (function r(cntr) { - if (--cntr <= 0) + if (cntr-- <= 0) return bench.end(n); fs.realpath(relative_path, function() { r(cntr); @@ -37,7 +37,7 @@ function relativePath(n) { function resolvedPath(n) { (function r(cntr) { - if (--cntr <= 0) + if (cntr-- <= 0) return bench.end(n); fs.realpath(resolved_path, function() { r(cntr); |