benchmark: remove extra spacing in http options
authorBrendan Ashworth <brendan.ashworth@me.com>
Thu, 29 Jan 2015 06:14:57 +0000 (22:14 -0800)
committerJeremiah Senkpiel <fishrock123@rocketmail.com>
Fri, 30 Jan 2015 21:09:10 +0000 (16:09 -0500)
This commit removes the benchmark spacing modification in
`client-request-body.js` and `end-vs-write-end.js` which adds two spaces
to the end of some variables to make sure the lines line up.

The reason behind this is that its totally pointless (the lines don't
actually line up with it) and it disallows you to parse the output with
a tool like awk, or at least makes it a lot harder.

PR-URL: https://github.com/iojs/io.js/pull/650
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
benchmark/http/client-request-body.js
benchmark/http/end-vs-write-end.js

index 7a3468a..1c27dea 100644 (file)
@@ -7,7 +7,7 @@ var bench = common.createBenchmark(main, {
   dur: [5],
   type: ['asc', 'utf', 'buf'],
   bytes: [32, 256, 1024],
-  method: ['write', 'end  '] // two spaces added to line up each row
+  method: ['write', 'end']
 });
 
 function main(conf) {
index 973f23a..c17e2ff 100644 (file)
@@ -14,7 +14,7 @@ var bench = common.createBenchmark(main, {
   type: ['asc', 'utf', 'buf'],
   kb: [64, 128, 256, 1024],
   c: [100],
-  method: ['write', 'end  '] // two spaces added to line up each row
+  method: ['write', 'end']
 });
 
 function main(conf) {