-
-
Notifications
You must be signed in to change notification settings - Fork 824
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
Some cleanup and refactor #398
Conversation
This is just a split commit, refraining from renaming too much. The drop(tx) call is no longer necessary, as the first sender is dropped at the end of spawn_senders(..)
It looks like the Travis checks are failing due to setup issues, not the PR per se. |
|
||
// Drop the initial sender. If we don't do this, the receiver will block even | ||
// if all threads have finished, since there is still one sender around. | ||
drop(tx); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I understand this correctly, calling drop
explicitly is not needed anymore because tx
is moved into the spawn_senders
function and dropped at the end of the function scope?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is correct. It's explained in the commit message of a05312e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, perfect! I missed that, sorry.
Awesome work, thank you very much! |
Not a complete form, but this beings some cleanup and function splitting mentioned in #382
This is not a very focused set of changes, just picking some low hanging refactoring fruit.