low_speed_limit: improved function for longer time periods #1390

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@bagder
Owner
bagder commented Apr 4, 2017

Reported-by: Paul Harris
Fixes #1345

@bagder, thanks for your PR! By analyzing the history of the files in this pull request, we identified @sylvestre, @yangtse and @dfandrich to be potential reviewers.

bagder added some commits Apr 4, 2017
@bagder bagder low_speed_limit: improved function for longer time periods
Reported-by: Paul Harris
Fixes #1345
07d0911
@bagder bagder test1606: verify speedcheck
56119e6
Owner
bagder commented Apr 6, 2017 edited

Fixed a minor issue where > is now >= and I added a unit test to verify the function.

@bagder bagder added a commit that closed this pull request Apr 6, 2017
@bagder bagder low_speed_limit: improved function for longer time periods
Previously, periods of fast speed between periods of slow speed would
not count and could still erroneously trigger a timeout.

Reported-by: Paul Harris
Fixes #1345
Closes #1390
2d5711d
@bagder bagder closed this in 2d5711d Apr 6, 2017
@bagder bagder deleted the bagder/low_speed_limit branch Apr 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment