Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Syntax error #222

Merged
merged 1 commit into from
Mar 30, 2012
Merged

Syntax error #222

merged 1 commit into from
Mar 30, 2012

Conversation

subnetmarco
Copy link
Contributor

Fixed syntax error.

@jfhbrook
Copy link
Contributor

Syntax error? This doesn't look like something that would break code..? I could be wrong? Seems more like a stylistic faux pax than anything.

Marak added a commit that referenced this pull request Mar 30, 2012
@Marak Marak merged commit 1a014f9 into http-party:master Mar 30, 2012
@Marak
Copy link
Contributor

Marak commented Mar 30, 2012

Merged thanks!

@subnetmarco
Copy link
Contributor Author

Actually there was a hidden char after target. If you opened it in Eclipse (UTF-8) it showed this.target  = {};

It is just a small fix, probably it was harmless but there's no reason to keep it in the code.

@jfhbrook
Copy link
Contributor

Actually there was a hidden char after target.

Whoa! That's bizarre.

It is just a small fix, probably it was harmless but there's no reason to keep it in the code.

Oh, I didn't meant to, like, defend the current code or anything. Marak just beat me to pulling this in. Really I agree with him; I just talk too much. ;)

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants