Skip to content

Changing gcnhub to use select() instead of threading, which should fix the problems with ssl#6

Open
sonicrules1234 wants to merge 4 commits into
KermMartian:masterfrom
sonicrules1234:master
Open

Changing gcnhub to use select() instead of threading, which should fix the problems with ssl#6
sonicrules1234 wants to merge 4 commits into
KermMartian:masterfrom
sonicrules1234:master

Conversation

@sonicrules1234
Copy link
Copy Markdown
Contributor

No description provided.

@tari
Copy link
Copy Markdown

tari commented Oct 26, 2014

A few observations from my end.

  • asyncore or asynchat can probably further cut down on the amount of code required, and is probably more efficient when mechanisms like epoll are available.
  • Pointing out that my Twisted rewrite exists too, though I ripped out a lot of messy functionality that wasn't in scope at the time I was working on it.

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.

2 participants