Skip to content

Add missing examples for std::cell types - #34736

Merged
bors merged 1 commit into
rust-lang:masterfrom
GuillaumeGomez:cells_doc
Jul 12, 2016
Merged

Add missing examples for std::cell types#34736
bors merged 1 commit into
rust-lang:masterfrom
GuillaumeGomez:cells_doc

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Member

Comment threadsrc/libcore/cell.rs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can this just be

assert_eq!(c.get(),6);

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Sure!

@GuillaumeGomez
GuillaumeGomezforce-pushed the cells_doc branch 2 times, most recently from 3ce6191 to 2c964e0CompareJuly 11, 2016 12:28
Comment threadsrc/libcore/cell.rs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can this block just be replaced with

*c.get_mut() += 1;

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Sure!

@GuillaumeGomez

Copy link
Copy Markdown
MemberAuthor

Updated.

@steveklabnik

Copy link
Copy Markdown
Contributor

@bors: r+ rollup

@bors

bors commented Jul 11, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 76dd020 has been approved by steveklabnik

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jul 12, 2016
bors added a commit that referenced this pull request Jul 12, 2016
Rollup of 7 pull requests
- Successful merges: #34736, #34737, #34740, #34742, #34749, #34750, #34770
- Failed merges: #33951
@bors
bors merged commit 76dd020 into rust-lang:masterJul 12, 2016
@GuillaumeGomez
GuillaumeGomez deleted the cells_doc branch July 12, 2016 14:07
Sign up for freeto 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.

4 participants

@GuillaumeGomez@steveklabnik@bors@apasel422