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

Already on GitHub? Sign in to your account

server module overhaul #1

Merged
merged 17 commits into from Jan 18, 2013

Conversation

Projects
None yet
2 participants

I pulled all of those closure literals out and documented a little bit. An EventEmitter becomes the this of its callbacks, so I figured we could take advantage of that and avoid closing over socket so much.

oops, made a bunch of other changes and didn't realize I can only have one pull request open at a time. I guess I should have made more branches

vantreeseba added a commit that referenced this pull request Jan 18, 2013

@vantreeseba vantreeseba merged commit 297930b into okcjs:master Jan 18, 2013

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