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.

Sign up for GitHub

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

Jump to bottom

doc: Socket.prototype.connect(options[, connectListener]) #951

Closed
wants to merge 1 commit into from
Closed

doc: Socket.prototype.connect(options[, connectListener]) #951

wants to merge 1 commit into from

Commits on Mar 1, 2015

  1. doc: Clean up net.Socket 

    net.Socket::connect(options[, connectListener]) was missing, with the
relevant
details found on the net.connect function. I moved the appropriate documentation
over and then rewrote the documentation for the function to say that it just
creates a socket and calls the connect method on it. I also changed the other
net.Socket::connect variants to say they are like the options version but
called with a specific options object.

net.connect and other functions were called methods even though they don't use
the `this` binding, so I changed method to function where appropriate.

Finally, I added a missing period to the end of the module summary. It's not
really related to the rest of the changes, but benjamingr noticed it.
    @Havvy
    Havvy committed Mar 1, 2015
    Configuration menu
    Copy the full SHA
    3100924 View commit details
    Browse the repository at this point in the history