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.

Sign up for GitHub

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

paper of timer wheel #2537

Closed
zyctree opened this issue May 15, 2020 · 1 comment · Fixed by #2572
Closed

paper of timer wheel #2537

zyctree opened this issue May 15, 2020 · 1 comment · Fixed by #2572
Labels
A-tokio Area: The main tokio crate C-question User questions that are neither feature requests nor bug reports M-time Module: tokio/time T-docs Topic: documentation

Comments

@zyctree
Copy link
Contributor

zyctree commented May 15, 2020

i would like to read the code about the detail implementation of timer, found

///
/// # Implementation
///
/// THe time driver is based on the [paper by Varghese and Lauck][paper].
///

but the link to the paper is missing, searching in history, there is a commit that removed this link,
7b53b7b#diff-e7962099c7699aa4d4efb9ac594f0665L82. is there any reason it was removed? seems like it was removed by accident

Btw, this link missing makes cargo doc --document-private-items fail
@Darksonn Darksonn added A-tokio Area: The main tokio crate C-question User questions that are neither feature requests nor bug reports M-time Module: tokio/time T-docs Topic: documentation labels May 15, 2020
@Darksonn
Copy link
Contributor

You can find the paper here. This issue will be fixed by #2572.

@MikailBag MikailBag mentioned this issue Jun 11, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-tokio Area: The main tokio crate C-question User questions that are neither feature requests nor bug reports M-time Module: tokio/time T-docs Topic: documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update doc comments MikailBag/tokio
2 participants
@Darksonn @zyctree