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

Improve primitive integers documentation #34709

Merged
merged 1 commit into from Jul 9, 2016

Conversation

GuillaumeGomez
Copy link
Member

Fixes #29335.

r? @steveklabnik

@@ -307,6 +309,8 @@ macro_rules! int_impl {
/// wrapping the truncated bits to the beginning of the resulting
/// integer.
///
/// Please note this isn't the same operation than `>>`!
Copy link

Choose a reason for hiding this comment

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

Replace "than" with "as"

@steveklabnik
Copy link
Member

While this is certainly an improvement, I don't think it fixes the issue. Not every single method has an example yet, for instance.

@steveklabnik
Copy link
Member

(that doesn't mean we shouldn't merge it, but the "Fixes" should get removed from the commit message/description)

@GuillaumeGomez
Copy link
Member Author

Then wait for what comes next!

@GuillaumeGomez
Copy link
Member Author

Updated.

///
/// # Examples
///
/// Basic usage:
Copy link
Member

Choose a reason for hiding this comment

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

We didn't accept this as a standard, so these shouldn't be added.

Copy link
Member Author

Choose a reason for hiding this comment

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

I did it to keep file consistency. The RFC isn't merged yet so for now I prefer to make it like other doc comments in this file. However, I can update other if you want.

Copy link
Member

Choose a reason for hiding this comment

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

I would rather not add more, as I'm gonna have to go back and remove them soon, and this just creates more work 😄

Copy link
Member Author

Choose a reason for hiding this comment

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

Then sure! ;)

@GuillaumeGomez
Copy link
Member Author

Updated.

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Jul 8, 2016

📌 Commit a6bbd0c has been approved by steveklabnik

@bors
Copy link
Contributor

bors commented Jul 9, 2016

⌛ Testing commit a6bbd0c with merge 459b1a4...

bors added a commit that referenced this pull request Jul 9, 2016
Improve primitive integers documentation

Fixes #29335.

r? @steveklabnik
@bors bors merged commit a6bbd0c into rust-lang:master Jul 9, 2016
@GuillaumeGomez GuillaumeGomez deleted the primitives branch July 9, 2016 23:01
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