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

Jump to bottom

SockServ redesign to avoid task #226

Open
nkolban opened this issue Nov 25, 2017 · 1 comment
Open

SockServ redesign to avoid task #226

nkolban opened this issue Nov 25, 2017 · 1 comment
Assignees
@nkolban
Labels
enhancement study required

Comments

@nkolban
Copy link
Owner

nkolban commented Nov 25, 2017

Study of the SockServ class is leaving me puzzled. It creates a task that seems to have no purpose other than to wait on accepts. Even though I wrote this class, I can see no value or purpose on this. I am also starting to sense timing issues and other poor considerations. This issue asks for the reworks of this class.
nkolban added a commit that referenced this issue Nov 25, 2017
@nkolban
base line for #226
5fafd44
@nkolban
Copy link
Owner Author

nkolban commented Nov 25, 2017

We'll start with a base line and then rework.

@nkolban nkolban self-assigned this Nov 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nkolban nkolban

Labels
enhancement study required
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nkolban