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

Stabilize some Duration consuming methods #27655

Closed

Conversation

sfackler
Copy link
Member

These methods are simply the Duration consuming variants of existing
stable methods that take a u32 of milliseconds.

r? @alexcrichton

Closes #27771

@sfackler sfackler added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Aug 11, 2015
@alexcrichton alexcrichton added the I-needs-decision Issue: In need of a decision. label Aug 11, 2015
@alexcrichton
Copy link
Member

👍

@sfackler
Copy link
Member Author

We might want to backport this to 1.3 as well, under the justification that these are the "super obvious" APIs to stabilize along with Duration.

@sfackler
Copy link
Member Author

One thing to consider is if Condvar::wait_timeout should return a bool or a more descriptive enum. I can never remember what value means "timeout" and which doesn't.

@sfackler sfackler force-pushed the duration-stabilization-followup branch 2 times, most recently from 3246dc8 to da36099 Compare August 11, 2015 14:37
These methods are simply the Duration consuming variants of existing
stable methods that take a u32 of milliseconds.
@sfackler sfackler force-pushed the duration-stabilization-followup branch from da36099 to 1d60cb7 Compare August 13, 2015 16:22
@sfackler
Copy link
Member Author

Updated to remove the Condvar::wait_timeout stabilization.

@alexcrichton
Copy link
Member

I'm going to close this in favor of the FCP issue on the tracker (see more info here: #27835 (comment)).

We'll tackle this towards the end of the cycle where we handle a number of FCP unstable features.

@sfackler sfackler deleted the duration-stabilization-followup branch November 26, 2016 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-needs-decision Issue: In need of a decision. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants