Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Cleaned up a couple of leaked variables #77

Merged
merged 2 commits into from

2 participants

@mmoulton

Some leaked variables were causing my test suite to complain, so I cleaned them up in these commits.

@chriso
Owner

Thanks

@chriso chriso merged commit 114b9a0 into chriso:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 9, 2012
  1. @mmoulton
  2. @mmoulton
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +1 −1  lib/node.io/dom.js
  2. +1 −1  lib/node.io/processor.js
View
2  lib/node.io/dom.js
@@ -47,7 +47,7 @@ Job.prototype.$ = function (selector, context, suppressErrors) {
*/
Job.prototype.parseHtml = function (data, callback, response) {
var self = this, recurse = this.options.recurse;
- headers = response && response.headers ? response.headers : {};
+ var headers = response && response.headers ? response.headers : {};
if (this.options.jsdom) {
var features = {
FetchExternalResources: this.options.external_resources,
View
2  lib/node.io/processor.js
@@ -11,7 +11,7 @@ var fs = require('fs'),
Job = require('./job').JobProto;
//Keep track of processes
-var isMaster, fork, distribute,
+var isMaster, isSlave, fork, distribute,
master, workers = [], worker_count = 0;
/**
Something went wrong with that request. Please try again.