Fix global leaks

ry · ry · commit 92789b16e594 · 2010-12-04T16:36:21.000-08:00
diff --git a/lib/net.js b/lib/net.js
@@ -853,7 +853,7 @@ Server.prototype._rejectPending = function() {
   // Accept and close the waiting clients one at a time.
   // Single threaded programming ftw.
   while (true) {
-    peerInfo = accept(this.fd);
+    var peerInfo = accept(this.fd);
     if (!peerInfo) return;
     close(peerInfo.fd);
 
diff --git a/test/fixtures/print-chars-from-buffer.js b/test/fixtures/print-chars-from-buffer.js
@@ -1,10 +1,9 @@
 var common = require('../common');
 var assert = require('assert');;
-Buffer = require('buffer').Buffer;
 
 var n = parseInt(process.argv[2]);
 
-b = new Buffer(n);
+var b = new Buffer(n);
 for (var i = 0; i < n; i++) {
   b[i] = 100;
 }
diff --git a/test/message/2100bytes.js b/test/message/2100bytes.js
@@ -1,7 +1,7 @@
-common = require('../common');
-assert = common.assert;
+var common = require('../common');
+var assert = require('assert');
+var util = require('util');
 
-util = require('util');
 console.log([
   '_______________________________________________50',
   '______________________________________________100',
diff --git a/test/message/hello_world.js b/test/message/hello_world.js
@@ -1,4 +1,4 @@
-common = require('../common');
-assert = common.assert;
+var common = require('../common');
+var assert = require('assert');
 
 console.log('hello world');
diff --git a/test/pummel/test-keep-alive.js b/test/pummel/test-keep-alive.js
@@ -1,11 +1,11 @@
 // This test requires the program 'ab'
 var common = require('../common');
 var assert = require('assert');;
-http = require('http');
-exec = require('child_process').exec;
+var http = require('http');
+var exec = require('child_process').exec;
 
-body = 'hello world\n';
-server = http.createServer(function(req, res) {
+var body = 'hello world\n';
+var server = http.createServer(function(req, res) {
   res.writeHead(200, {
     'Content-Length': body.length,
     'Content-Type': 'text/plain'
diff --git a/test/pummel/test-net-many-clients.js b/test/pummel/test-net-many-clients.js
@@ -1,6 +1,7 @@
 var common = require('../common');
 var assert = require('assert');;
-net = require('net');
+var net = require('net');
+
 // settings
 var bytes = 1024 * 40;
 var concurrency = 100;
diff --git a/test/pummel/test-net-pause.js b/test/pummel/test-net-pause.js
@@ -1,10 +1,11 @@
 var common = require('../common');
 var assert = common.assert;
 var net = require('net');
+
 var N = 200;
 var recv = '', chars_recved = 0;
 
-server = net.createServer(function(connection) {
+var server = net.createServer(function(connection) {
   function write(j) {
     if (j >= N) {
       connection.end();
@@ -17,8 +18,9 @@ server = net.createServer(function(connection) {
   }
   write(0);
 });
+
 server.on('listening', function() {
-  client = net.createConnection(common.PORT);
+  var client = net.createConnection(common.PORT);
   client.setEncoding('ascii');
   client.addListener('data', function(d) {
     common.print(d);
diff --git a/test/pummel/test-net-pingpong-delay.js b/test/pummel/test-net-pingpong-delay.js
@@ -1,6 +1,6 @@
 var common = require('../common');
 var assert = require('assert');;
-net = require('net');
+var net = require('net');
 
 
 var tests_run = 0;
diff --git a/test/pummel/test-net-pingpong.js b/test/pummel/test-net-pingpong.js
@@ -1,6 +1,6 @@
 var common = require('../common');
 var assert = require('assert');;
-net = require('net');
+var net = require('net');
 
 var tests_run = 0;
 
diff --git a/test/pummel/test-net-throttle.js b/test/pummel/test-net-throttle.js
@@ -1,11 +1,10 @@
 var common = require('../common');
 var assert = require('assert');;
-net = require('net');
-N = 160 * 1024; // 30kb
+var net = require('net');
 
-
-chars_recved = 0;
-npauses = 0;
+var N = 160 * 1024; // 30kb
+var chars_recved = 0;
+var npauses = 0;
 
 console.log('build big string');
 var body = '';
@@ -15,15 +14,16 @@ for (var i = 0; i < N; i++) {
 
 console.log('start server on port ' + common.PORT);
 
-server = net.createServer(function(connection) {
+var server = net.createServer(function(connection) {
   connection.addListener('connect', function() {
     assert.equal(false, connection.write(body));
     connection.end();
   });
 });
+
 server.listen(common.PORT, function() {
   var paused = false;
-  client = net.createConnection(common.PORT);
+  var client = net.createConnection(common.PORT);
   client.setEncoding('ascii');
   client.addListener('data', function(d) {
     chars_recved += d.length;
@@ -33,7 +33,7 @@ server.listen(common.PORT, function() {
       npauses += 1;
       paused = true;
       console.log('pause');
-      x = chars_recved;
+      var x = chars_recved;
       setTimeout(function() {
         assert.equal(chars_recved, x);
         client.resume();
diff --git a/test/pummel/test-net-timeout.js b/test/pummel/test-net-timeout.js
@@ -1,10 +1,11 @@
 var common = require('../common');
 var assert = require('assert');;
-net = require('net');
-exchanges = 0;
-starttime = null;
-timeouttime = null;
-timeout = 1000;
+var net = require('net');
+
+var exchanges = 0;
+var starttime = null;
+var timeouttime = null;
+var timeout = 1000;
 
 var echo_server = net.createServer(function(socket) {
   socket.setTimeout(timeout);
diff --git a/test/pummel/test-timers.js b/test/pummel/test-timers.js
@@ -1,8 +1,6 @@
 var common = require('../common');
 var assert = require('assert');
 
-assert = require('assert');
-
 var WINDOW = 200; // why is does this need to be so big?
 
 var interval_count = 0;
@@ -79,8 +77,8 @@ setInterval(function(param1, param2){
 }, 1000, 'param1', 'param2');
 
 // setInterval(cb, 0) should be called multiple times.
-count4 = 0;
-interval4 = setInterval(function() {
+var count4 = 0;
+var interval4 = setInterval(function() {
   if (++count4 > 10) clearInterval(interval4);
 }, 0);
 
@@ -92,12 +90,12 @@ function t () {
   expectedTimeouts--;
 }
 
-w = setTimeout(t, 200),
-x = setTimeout(t, 200),
-y = setTimeout(t, 200);
+var w = setTimeout(t, 200);
+var x = setTimeout(t, 200);
+var y = setTimeout(t, 200);
 
-clearTimeout(y),
-z = setTimeout(t, 200);
+clearTimeout(y);
+var z = setTimeout(t, 200);
 clearTimeout(y);
 
 
diff --git a/test/simple/test-http-exceptions.js b/test/simple/test-http-exceptions.js
@@ -2,7 +2,7 @@ var common = require('../common');
 var assert = require('assert');;
 var http = require('http');
 
-server = http.createServer(function (req, res) {
+var server = http.createServer(function (req, res) {
   intentionally_not_defined();
   res.writeHead(200, {"Content-Type": "text/plain"});
   res.write("Thank you, come again.");
@@ -17,7 +17,7 @@ server.listen(common.PORT, function () {
   }
 });
 
-exception_count = 0;
+var exception_count = 0;
 
 process.addListener("uncaughtException", function (err) {
   console.log("Caught an exception: " + err);
diff --git a/test/simple/test-http-expect-continue.js b/test/simple/test-http-expect-continue.js
@@ -1,5 +1,5 @@
 var common = require("../common");
-var assert = common.assert;
+var assert = require('assert');
 var http = require("http");
 
 var outstanding_reqs = 0;
@@ -31,12 +31,12 @@ server.listen(common.PORT);
 
 server.addListener("listening", function() {
   var client = http.createClient(common.PORT);
-  req = client.request("POST", "/world", {
+  var req = client.request("POST", "/world", {
       "Expect": "100-continue",
   });
   common.debug("Client sending request...");
   outstanding_reqs++;
-  body = "";
+  var body = "";
   req.addListener('continue', function () {
       common.debug("Client got 100 Continue...");
       got_continue = true;
diff --git a/test/simple/test-http-keep-alive-close-on-header.js b/test/simple/test-http-keep-alive-close-on-header.js
@@ -1,20 +1,18 @@
 var common = require('../common');
 var assert = require('assert');
-
-assert = require("assert");
 var http = require('http');
 var util = require('util');
 
-body = "hello world\n";
-headers = {'connection':'keep-alive'}
+var body = "hello world\n";
+var headers = {'connection':'keep-alive'}
 
-server = http.createServer(function (req, res) {
+var server = http.createServer(function (req, res) {
   res.writeHead(200, {"Content-Length": body.length, "Connection":"close"});
   res.write(body);
   res.end();
 });
 
-connectCount = 0;
+var connectCount = 0;
 
 server.listen(common.PORT, function () {
   var client = http.createClient(common.PORT);
diff --git a/test/simple/test-http-keep-alive.js b/test/simple/test-http-keep-alive.js
@@ -1,20 +1,18 @@
 var common = require('../common');
 var assert = require('assert');
-
-assert = require("assert");
 var http = require('http');
 var util = require('util');
 
-body = "hello world\n";
-headers = {'connection':'keep-alive'}
+var body = "hello world\n";
+var headers = {'connection':'keep-alive'}
 
-server = http.createServer(function (req, res) {
+var server = http.createServer(function (req, res) {
   res.writeHead(200, {"Content-Length": body.length});
   res.write(body);
   res.end();
 });
 
-connectCount = 0;
+var connectCount = 0;
 
 server.listen(common.PORT, function () {
   var client = http.createClient(common.PORT);
diff --git a/test/simple/test-http-server.js b/test/simple/test-http-server.js
@@ -2,8 +2,8 @@ var common = require('../common');
 var assert = require('assert');
 var net = require('net');
 var http = require('http');
-url = require("url");
-qs = require("querystring");
+var url = require("url");
+var qs = require("querystring");
 
 var request_number = 0;
 var requests_sent = 0;
diff --git a/test/simple/test-http-upgrade-server2.js b/test/simple/test-http-upgrade-server2.js
@@ -1,10 +1,9 @@
 var common = require('../common');
 var assert = require('assert');
-
 var http = require('http');
 var net = require('net');
 
-server = http.createServer(function (req, res) {
+var server = http.createServer(function (req, res) {
   common.error('got req');
   throw new Error("This shouldn't happen.");
 });
@@ -16,7 +15,7 @@ server.addListener('upgrade', function (req, socket, upgradeHead) {
   throw new Error('upgrade error');
 });
 
-gotError = false;
+var gotError = false;
 
 process.addListener('uncaughtException', function (e) {
   common.error('got "clientError" event');
diff --git a/test/simple/test-http-write-empty-string.js b/test/simple/test-http-write-empty-string.js
@@ -2,9 +2,8 @@ var common = require('../common');
 var assert = require('assert');
 
 var http = require('http');
-assert = require('assert');
 
-server = http.createServer(function (request, response) {
+var server = http.createServer(function (request, response) {
   console.log('responding to ' + request.url);
 
   response.writeHead(200, {'Content-Type': 'text/plain'});
diff --git a/test/simple/test-http.js b/test/simple/test-http.js
@@ -1,7 +1,7 @@
 var common = require('../common');
 var assert = require('assert');
 var http = require('http');
-url = require("url");
+var url = require("url");
 
 function p (x) {
   common.error(common.inspect(x));
diff --git a/test/simple/test-listen-fd.js b/test/simple/test-listen-fd.js
@@ -4,7 +4,7 @@
 var common = require('../common');
 var assert = require('assert');
 var http = require('http');
-netBinding = process.binding('net');
+var netBinding = process.binding('net');
 
 // Create an server and set it listening on a socket bound to common.PORT
 var gotRequest = false;
diff --git a/test/simple/test-net-binary.js b/test/simple/test-net-binary.js
@@ -2,10 +2,10 @@ var common = require('../common');
 var assert = require('assert');
 var net = require('net');
 
-binaryString = "";
+var binaryString = "";
 for (var i = 255; i >= 0; i--) {
   var s = "'\\" + i.toString(8) + "'";
-  S = eval(s);
+  var S = eval(s);
   common.error( s
        + " "
        + JSON.stringify(S)
diff --git a/test/simple/test-net-server-max-connections.js b/test/simple/test-net-server-max-connections.js
@@ -8,12 +8,12 @@ var net = require('net');
 // TODO: test that the server can accept more connections after it reaches
 // its maximum and some are closed.
 
-N = 200;
-count = 0;
-closes = 0;
-waits = [];
+var N = 200;
+var count = 0;
+var closes = 0;
+var waits = [];
 
-server = net.createServer(function (connection) {
+var server = net.createServer(function (connection) {
   console.error("connect %d", count++);
   connection.write("hello");
   waits.push(function () { connection.end(); });
diff --git a/test/simple/test-pipe-head.js b/test/simple/test-pipe-head.js
diff --git a/test/simple/test-pump-file2tcp-noexist.js b/test/simple/test-pump-file2tcp-noexist.js
diff --git a/test/simple/test-pump-file2tcp.js b/test/simple/test-pump-file2tcp.js
diff --git a/test/simple/test-signal-handler.js b/test/simple/test-signal-handler.js
diff --git a/test/simple/test-stdin-from-file.js b/test/simple/test-stdin-from-file.js
diff --git a/test/simple/test-stdout-to-file.js b/test/simple/test-stdout-to-file.js
diff --git a/test/simple/test-string-decoder.js b/test/simple/test-string-decoder.js
