Minor jshint fixes

qunitjs · Aug 9, 2015 · fbfbc4b · fbfbc4b
1 parent 1f9698a
commit fbfbc4b
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/.jshintrc b/.jshintrc
@@ -12,5 +12,9 @@
   "sub": true,
   "trailing": true,
   "undef": true,
-  "unused": true
+  "unused": true,
+  "predef": [
+      "require",
+      "global"
+  ]
 }
diff --git a/bin/cli.js b/bin/cli.js
@@ -233,7 +233,7 @@ function attachWorkerHelpers(worker) {
 
       // worker has not acknowledged itself in 60 sec, reopen url
       client.changeUrl(self.id, { url: self.buildUrl() }, function () {
-        logger.debug("[%s] Sent Request to reload url", self.getTestBrowserInfo());
+        logger.debug('[%s] Sent Request to reload url', self.getTestBrowserInfo());
       });
 
     }, ackTimeout * 1000);

diff --git a/lib/proxy.js b/lib/proxy.js
@@ -4,7 +4,6 @@ var http = require('http'),
 var ProxyServer = {
   onRequest: function(client_req, client_res, host, callback) {
     var proxyUrl = url.parse(host);
-    var path = url.parse(host);
     var options = {
       path: client_req.url,
       hostname: proxyUrl.hostname,
@@ -14,7 +13,7 @@ var ProxyServer = {
     };
 
     var proxy = http.request(options, function (res) {
-      data = "";
+      var data = '';
       res.on('data', function(chunk) {
         data += chunk;
       });

diff --git a/lib/utils.js b/lib/utils.js
@@ -7,8 +7,6 @@ var Log = require('./logger'),
 
 String.prototype.escapeSpecialChars = function() {
   return this.replace(/\n/g, '\\n')
-  // TODO what is this supposed to do? JSHint considers this "Bad or unnecessary escaping."
-  .replace(/\\s/g, '\s')
   .replace(/\\\'/, '\'');
 };