Open
Conversation
Reworked some of the network calls to newer, more robust functions. Not strictly necessary for ipv6 support but I've been learning them so there you go...
The user methods will handle current logged in users. This will later on be expanded to handle saving users to some permenant storage. Since logged in users will be synonymous with threads, there is a thread handle method where the thread will spend most of its time. The message queue interface is how the threads talk to each other. There will be plenty of times that threads need to talk to each other, for example if someone sends a 'tell' to another user or a simple broadcast message. Threads should exclusivly handle input and output to their network socket, but anyone can write to their queue.
Ok so, mqueues are very much not portable. They're not on osx. Pipes are much more universal, but I don't yet know all the drawbacks. Renamed the messaging infrastructure to be more generic, in case we change the backend mechanism in the future.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.