Skip to content
This repository has been archived by the owner on Feb 10, 2024. It is now read-only.

PR 1320 #1631

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

PR 1320 #1631

wants to merge 2 commits into from

Conversation

Arnavion
Copy link
Contributor

@pstratem
fix timeout logic
085d9c3 
remove broken hex_net_ping_timeout logic

use timestamps in lag_check instead of relying on the function being called every 30 seconds

use prefs.hex_net_ping_timeout if available

fix whitespace

fix whitespace
@pstratem
run lagcheck even if gui is disabled
ca2b4d4
@Arnavion Arnavion mentioned this pull request Mar 13, 2016
Closed
@DerekTurtleRoe
Copy link

@Arnavion @pstratem So what does this do? The title isn't very informative and the code changes are a little confusing.

This obviously affects the calculation of the lag, but what does that fix in itself?

Also, does timeout have to occur only when the lag is bad? It also occurs after long periods of inactivity, doesn't it?

@Arnavion
Copy link
Contributor Author

I've added the link to original discussion.

@DerekTurtleRoe
Copy link

@Arnavion OK, thanks. I didn't know about the link, sorry.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Arnavion @DerekTurtleRoe @pstratem