Fix leak and multiply defined variable #14

Merged
merged 1 commit into from Jan 8, 2013

Conversation

Projects
None yet
2 participants
Contributor

jessejlt commented Dec 9, 2012

Variable prop was leaking, and property was multiply defined and was being used outside of various scopes.  Also adding some missing semicolons to make my linter happy.

Fix leak and multiply defined variable
Variable `prop` was leaking, and `property` was multiply defined and was being used outside of various scopes. 
if (schema.properties !== undefined) {
- for (var property in schema.properties) {
+ for (property in schema.properties) {
prop = validateSchema(obj[property], schema.properties[property], names.concat([property]));
@jessejlt

jessejlt Dec 9, 2012

Contributor

Here's where the leak occurred.

oferei added a commit that referenced this pull request Jan 8, 2013

Merge pull request #14 from jessejlt/patch-1
Fix leak and multiply defined variable

@oferei oferei merged commit a0afe0d into oferei:master Jan 8, 2013

Owner

oferei commented Jan 8, 2013

thanks mate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment