Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Send() address and port should be args #31

Closed
marcoscaceres opened this issue Jun 2, 2013 · 2 comments
Closed

Send() address and port should be args #31

marcoscaceres opened this issue Jun 2, 2013 · 2 comments

Comments

@marcoscaceres
Copy link
Contributor

If address and port are required, then the should me arguments of send(), not options. Please only use options for optional things.

@ClaesNilsson
Copy link
Contributor

I am not sure what you mean. If you are talking about UDP then default remote address and port could optionally be set by the constructor. This means that if remote address and port is omitted in the send() method the default remote address and port are used as destination. If no remote address and port arguments are given in the send() method and no default values were set by the constructor this is an error.
Could above be described differently in the specification?

@marcoscaceres
Copy link
Contributor Author

I might return to this later, but send has already gone through some changes so this might not apply anymore. Hence, closing.

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

No branches or pull requests

2 participants