Skip to content

Socket improvements#36

Open
tcunha wants to merge 10 commits intoryanflannery:masterfrom
tcunha:socket
Open

Socket improvements#36
tcunha wants to merge 10 commits intoryanflannery:masterfrom
tcunha:socket

Conversation

@tcunha
Copy link
Copy Markdown
Contributor

@tcunha tcunha commented Jan 27, 2013

No description provided.

Tiago Cunha added 10 commits February 12, 2013 10:50
By adding missing options, alphabetically sort options and wording
consistency.
Before, it wouldn't be possible for two simultaneous users to start
vitunes and send messages through the socket.

Its path (defaults to ~/.vitunes/vitunes.sock), like the remaining ones,
can also be configured with the newly added -s command-line option.
In preference of, manually calculating it or possibly using SUN_LEN.
All cases are valid, because sun_path must be NUL terminated.
Imperative, now that the path (POSIX says its size is undefined) can be
user controlled. In case of truncation, set errno accordingly, which is
caught by the caller.
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.

1 participant