Minor style improvements and a few new comments

Author: driverdan

lib/XMLHttpRequest.js

@@ -11,9 +11,9 @@
  * @license MIT
  */
 
-var Url = require("url"),
-  spawn = require("child_process").spawn,
-  fs = require('fs');
+var Url = require("url")
+  , spawn = require("child_process").spawn
+  , fs = require('fs');
 
 exports.XMLHttpRequest = function() {
   /**
@@ -402,7 +402,7 @@ exports.XMLHttpRequest = function() {
       fs.writeFileSync(syncFile, "", "utf8");
       // The async request the other Node process executes
       var execString = "var http = require('http'), https = require('https'), fs = require('fs');"
-        + "var doRequest = http" + (ssl?"s":"") + ".request;"
+        + "var doRequest = http" + (ssl ? "s" : "") + ".request;"
         + "var options = " + JSON.stringify(options) + ";"
         + "var responseText = '';"
         + "var req = doRequest(options, function(response) {"
@@ -430,7 +430,7 @@ exports.XMLHttpRequest = function() {
       syncProc.stdin.end();
       // Remove the temporary file
       fs.unlinkSync(syncFile);
-      if(self.responseText.match(/^NODE-XMLHTTPREQUEST-ERROR:/)) {
+      if (self.responseText.match(/^NODE-XMLHTTPREQUEST-ERROR:/)) {
         // If the file returned an error, handle it
         var errorObj = self.responseText.replace(/^NODE-XMLHTTPREQUEST-ERROR:/, "");
         self.handleError(errorObj);
@@ -443,6 +443,9 @@ exports.XMLHttpRequest = function() {
     }
   };
 
+  /**
+   * Called when an error is encountered to deal with it.
+   */
   this.handleError = function(error) {
     this.status = 503;
     this.statusText = error;

tests/test-constants.js

@@ -1,7 +1,7 @@
 var sys = require("util")
-  ,assert = require("assert")
-  ,XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
-  ,xhr = new XMLHttpRequest();
+  , assert = require("assert")
+  , XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
+  , xhr = new XMLHttpRequest();
 
 // Test constant values
 assert.equal(0, xhr.UNSENT);

tests/test-exceptions.js

@@ -1,24 +1,7 @@
 var sys = require("util")
-  ,assert = require("assert")
-  ,XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
-  ,xhr = new XMLHttpRequest()
-  ,http = require("http");
-
-// Test server
-/*
-var server = http.createServer(function (req, res) {
-  var body = "Hello World";
-  res.writeHead(200, {
-    "Content-Type": "text/plain",
-    "Content-Length": Buffer.byteLength(body),
-    "Connection": "close"
-  });
-  res.write("Hello World");
-  res.end();
-
-  this.close();
-}).listen(8000);
-*/
+  , assert = require("assert")
+  , XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
+  , xhr = new XMLHttpRequest();
 
 // Test request methods that aren't allowed
 try {

tests/test-headers.js

@@ -1,8 +1,8 @@
 var sys = require("util")
-  ,assert = require("assert")
-  ,XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
-  ,xhr = new XMLHttpRequest()
-  ,http = require("http");
+  , assert = require("assert")
+  , XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
+  , xhr = new XMLHttpRequest()
+  , http = require("http");
 
 // Test server
 var server = http.createServer(function (req, res) {

tests/test-request-methods.js

@@ -1,8 +1,8 @@
 var sys = require("util")
-  ,assert = require("assert")
-  ,XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
-  ,http = require("http")
-  ,xhr;
+  , assert = require("assert")
+  , XMLHttpRequest = require("../lib/XMLHttpRequest").XMLHttpRequest
+  , http = require("http")
+  , xhr;
 
 // Test server
 var server = http.createServer(function (req, res) {