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

std: Add more docs for ptr mod #13386

Closed
wants to merge 1 commit into from
Closed

Conversation

brson
Copy link
Contributor

@brson brson commented Apr 7, 2014

No description provided.

//! ## 3. Get it from C.
//!
//! ```
//! use std::libc;
Copy link
Member

Choose a reason for hiding this comment

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

The libc extraction landed, so this should probably change to extern crate libc;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can't write extern crate here, which seems like a more general problem.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, if I put a main fn somewhere it's better.

@alexcrichton
Copy link
Member

mmmm delicious docs

r=me with the libc thing and a whitespace fix pointed out by travis

bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 18, 2022
Refactor completions expansion

Depends on rust-lang/rust-analyzer#13384
Diff is unfortunately massive as I changed the functions in the analysis module from associated ones to standalone (unfortunately without an extra commit)
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 24, 2024
…ason, r=xFrednet

Not trigger `duplicated_attributes` on duplicate reasons

As at rust-lang#13355 we shoudn't  trigger `duplicated_attributes` on duplicate reasons attr
changelog: [`duplicated_attributes`]: not trigger `duplicated_attributes` on duplicate reasons
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.

2 participants