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.

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

Implement Nik's Multithreading #1921

Closed
wants to merge 461 commits into from

Conversation

ThomasBeHappy
Copy link
Contributor

@ThomasBeHappy ThomasBeHappy commented Sep 7, 2021

#1610

Fulfilled the requested changes. Hopefully we can actually get this implemented sometime

valentinbreiz and others added 30 commits May 12, 2021 16:57
[Net] Implement TCP packet sequencing
Quick Typofix
Fix wrong 0:/ Documentation
AnErrupTion and others added 25 commits July 25, 2021 13:49
Update ConsoleImpl.cs to use exceptions
Several users found them annoying. They can be easily added manually by whoever wants them.
Remove before-after Run() debug lines
Add RemoteEndPoint and LocalEndPoint properties to TcpClient
Copy link

@Rhys-Woolcott Rhys-Woolcott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great.

@quajak quajak changed the base branch from master to impl-multitask September 9, 2021 07:04
@valentinbreiz
Copy link
Member

Hello @ThomasBeHappy, this can be kept for people who want to use your work on their own Cosmos implementation but this pull request won't be merged in master. As described in #1610, this multitask implementation is more a POC than a real thing, I close this.

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.