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

#21349: Refactor long functions in entrynodes.c (4) #246

Merged
merged 4 commits into from Jul 30, 2018
Merged

#21349: Refactor long functions in entrynodes.c (4) #246

merged 4 commits into from Jul 30, 2018

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@rl1987 @coveralls @tlyu @torproject-pusher
@rl1987
Copy link
Contributor

@rl1987 rl1987 commented Jul 21, 2018

https://trac.torproject.org/projects/tor/ticket/21349
rl1987 added 4 commits Jul 21, 2018
@rl1987
Split sampled_guards_update_from_consensus() into subfunctions
86549c0
@rl1987
Make entry_guards_update_primary() shorter
e6c51a0
@rl1987
Split select_entry_guard_for_circuit()
042d22c
@rl1987
Add changes file
e9f6f74
@coveralls
Copy link

@coveralls coveralls commented Jul 21, 2018

Coverage Status

Coverage increased (+0.01%) to 59.538% when pulling e9f6f74 on rl1987:ticket21349_4 into 9ae3597 on torproject:master.

@tlyu tlyu mentioned this pull request Jul 25, 2018
Closed
@tlyu tlyu mentioned this pull request Jul 25, 2018
Closed
@tlyu tlyu mentioned this pull request Jul 25, 2018
Closed
@tlyu
Copy link
Contributor

@tlyu tlyu commented Jul 25, 2018

Closed pull requests #173, #200, and #202 because this one supersedes them. These closed PRs contain some prior review comments.

@torproject-pusher torproject-pusher merged commit e9f6f74 into torproject:master Jul 30, 2018
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment