Skip to content

core: Inherit the atomics module #14169

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

Closed
wants to merge 4 commits into from

Conversation

alexcrichton
Copy link
Member

This module is a foundation on which many other algorithms are built. When hardware support is missing, stubs are provided in libcompiler-rt.a, so this should be available on all platforms.

//! `&fmt::Arguments`, a `&str`, and a `uint. These three arguments dictate
//! the failure message, the file at which failure was invoked, and the line.
//! It is up to consumers of this core library to define this failure
//! function, it is only required to never return.
Copy link
Member

Choose a reason for hiding this comment

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

the , should be either a ; or a ..

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated, thanks!

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Explain why it should not exist, and the plan of attack for removing it.

Verified

This commit was signed with the committer’s verified signature.
Due to our excellent macro hygiene, this involves having a global path and a
hidden module in libcore itself.
bors added a commit that referenced this pull request May 14, 2014

Verified

This commit was signed with the committer’s verified signature.
This module is a foundation on which many other algorithms are built. When hardware support is missing, stubs are provided in libcompiler-rt.a, so this should be available on all platforms.
@bors bors closed this May 14, 2014
@@ -9,6 +9,27 @@
// except according to those terms.

//! The Rust core library
//!
//! This library is meant to represent the core functionality of rust that is
//! maximally portable to other platforms. To that exent, this library has no
Copy link
Contributor

Choose a reason for hiding this comment

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

extent?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops, good catches! I'm piggy backing another PR to get these changes in.

@alexcrichton alexcrichton deleted the atomics branch May 14, 2014 15:55
flip1995 pushed a commit to flip1995/rust that referenced this pull request Mar 20, 2025
…less_late_init` (rust-lang#14169)

fixes rust-lang#9895

changelog: [`needless_late_init`]: correct suggestion when assignments
have enclosing parentheses
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.

None yet

4 participants