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

Add a few missing stability attributes #21745

Merged

Conversation

chris-morgan
Copy link
Member

I’d kind of like to be able to use HashState in AnyMap, which I can’t do without a stability attribute on it. While I was at it I looked around and found a few more missing.

Closes #21883.

@rust-highfive
Copy link
Collaborator

r? @huonw

(rust_highfive has picked a reviewer for you, use r? to override)

@@ -45,6 +45,7 @@ macro_rules! try_opt {

/// ISO 8601 time duration with nanosecond precision.
/// This also allows for the negative duration; see individual methods for details.
#[unstable(feature = "std_misc")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding #![feature(std_misc)] seems to already make this type available, did some code you have not compile with the feature gate?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was only HashState that I found myself unable to use; I didn’t try any of the rest. But still, they should have these markers, should they not?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah this was probably a vicitm of inheritance. An #[unstable] attribute is inherited by all children by default. The entire std::time::duration module is marked #[unstable] (which this inherits), but the hash_map module is not unstable and Hasher was just erroneously missing a marker.

@chris-morgan chris-morgan force-pushed the add-missing-unstable-attributes branch from 2f8d6b0 to 9836742 Compare January 30, 2015 04:22
@huonw
Copy link
Member

huonw commented Jan 30, 2015

r? @alexcrichton

@aturon
Copy link
Member

aturon commented Feb 3, 2015

Note: this is currently blocking Servo's rustup. cc @SimonSapin

@SimonSapin
Copy link
Contributor

Fixes #21883.

@huonw
Copy link
Member

huonw commented Feb 3, 2015

@bors r+ 9836 p=1

@bors
Copy link
Contributor

bors commented Feb 3, 2015

⌛ Testing commit 9836742 with merge cfc9109...

bors added a commit that referenced this pull request Feb 3, 2015
…r=huonw

I’d kind of like to be able to use HashState in AnyMap, which I can’t do without a stability attribute on it. While I was at it I looked around and found a few more missing.
@bors bors merged commit 9836742 into rust-lang:master Feb 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"there is no way to use this feature" for std::collections::hash_state::DefaultState
7 participants