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

IRC connection logic is broken #13

Closed
Chronial opened this issue May 1, 2016 · 1 comment
Closed

IRC connection logic is broken #13

Chronial opened this issue May 1, 2016 · 1 comment

Comments

@Chronial
Copy link
Collaborator

Chronial commented May 1, 2016

This code does not what it should:

If Val(txtCurrNodes.Text) < 4 Then

'Autoconnect to any nodes when Currnodes are under 4
If Val(txtCurrNodes.Text) < 4 Then
     attemptConnSteamID(tmpSteamID)

The problem is that connecting to nodes takes some time, so txtCurrNodes is actually a number from the past. The result is that for a few seconds, a new connection is initiated every 200 ms (refTimer interval). In my test, dscm initiated 40 connections after I activated dscm-net.

@Wulf2k
Copy link
Owner

Wulf2k commented May 1, 2016

I was aware of the issue but didn't think it would get that bad this
quickly. I'll build a connection queue tomorrow some time.
On May 1, 2016 17:49, "Chronial" [email protected] wrote:

This code does not what it should:

If Val(txtCurrNodes.Text) < 4 Then

'Autoconnect to any nodes when Currnodes are under 4If Val(txtCurrNodes.Text) < 4 Then
attemptConnSteamID(tmpSteamID)```

The problem is that connecting to nodes takes some time, so txtCurrNodes is actually a number from the past. The result is that for a few seconds, a new connection is initiated every 200 ms (refTimer interval). In my test, dscm initiated 40 connections after I activated dscm-net.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#13

@Chronial Chronial closed this as completed May 4, 2016
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

No branches or pull requests

2 participants