bench: fix buffers/buffer-base64-encode benchmark
authorBen Noordhuis <info@bnoordhuis.nl>
Wed, 4 Jun 2014 13:25:09 +0000 (15:25 +0200)
committerTrevor Norris <trev.norris@gmail.com>
Wed, 4 Jun 2014 21:37:27 +0000 (14:37 -0700)
The test is supposed to measure the performance of the base64 encoder
so move the Buffer#write() calls out of the benchmark section.

The overhead of the calls isn't terrible (about 1-3%) but having
unrelated activity in a micro-benchmark is never a good idea.

Signed-off-by: Trevor Norris <trev.norris@gmail.com>
benchmark/buffers/buffer-base64-encode.js

index f2b8e9a..3dcc13b 100644 (file)
@@ -28,9 +28,8 @@ function main(conf) {
   var b = Buffer(N);
   var s = '';
   for (var i = 0; i < 256; ++i) s += String.fromCharCode(i);
-
-  bench.start();
   for (var i = 0; i < N; i += 256) b.write(s, i, 256, 'ascii');
+  bench.start();
   for (var i = 0; i < 32; ++i) b.toString('base64');
   bench.end(64);
 }