benchmark: fix variables not being set

BridgeAR · MylesBorins · commit 7da01f43fdfc · 2018-02-21T10:18:53.000-05:00
Due to the destructuring the outer variables were not set anymore. PR-URL: #18320 Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/benchmark/tls/tls-connect.js b/benchmark/tls/tls-connect.js
@@ -11,12 +11,13 @@ const bench = common.createBenchmark(main,{
 
 var clientConn = 0;
 var serverConn = 0;
-var server;
 var dur;
 var concurrency;
 var running = true;
 
-function main({dur, concurrency }){
+function main(conf){
+ dur = conf.dur;
+ concurrency = conf.concurrency;
 const cert_dir = path.resolve(__dirname, '../../test/fixtures');
 const options ={
 key: fs.readFileSync(`${cert_dir}/test_key.pem`),
@@ -25,7 +26,7 @@ function main({dur, concurrency }){
 ciphers: 'AES256-GCM-SHA384'
 };
 
- server = tls.createServer(options, onConnection);
+ const server = tls.createServer(options, onConnection);
 server.listen(common.PORT, onListening);
 }
 

-Original file line number
+Diff line change
 varclientConn=0;
 varserverConn=0;
 -varserver;
 vardur;
 varconcurrency;
 varrunning=true;
 -functionmain({ dur, concurrency }){
 +functionmain(conf){
 +dur=conf.dur;
 +concurrency=conf.concurrency;
 constcert_dir=path.resolve(__dirname,'../../test/fixtures');
 constoptions={
 key: fs.readFileSync(`${cert_dir}/test_key.pem`),
 ciphers: 'AES256-GCM-SHA384'
 };
 -server=tls.createServer(options,onConnection);
 +constserver=tls.createServer(options,onConnection);
 server.listen(common.PORT,onListening);
+}