Skip to content

Add missing urls for AtomicPtr #38635

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Dec 30, 2016
Merged

Add missing urls for AtomicPtr #38635

merged 1 commit into from
Dec 30, 2016

Conversation

GuillaumeGomez
Copy link
Member

frewsxcv
frewsxcv reviewed Dec 27, 2016
View reviewed changes
src/libcore/sync/atomic.rs
/// ordering of this operation. The first describes the required ordering if the operation
/// succeeds while the second describes the required ordering when the operation fails. The
/// failure ordering can't be `Release` or `AcqRel` and must be equivalent or weaker than the
/// success ordering.
/// failure ordering can't be [`Release`] or [`AcqRel`] and must be equivalent or
Copy link
Member

Choose a reason for hiding this comment

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

Where are these links linking to?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh damn.

@GuillaumeGomez
Copy link
Member Author

Updated.

@frewsxcv
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 28, 2016

📌 Commit ef8d5e1 has been approved by frewsxcv

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 30, 2016
@alexcrichton
Rollup merge of rust-lang#38635 - GuillaumeGomez:atomicptr_docs, r=fr…
8b8ab85 
…ewsxcv

Add missing urls for AtomicPtr

r? @frewsxcv
@alexcrichton alexcrichton mentioned this pull request Dec 30, 2016
Merged
bors added a commit that referenced this pull request Dec 30, 2016
@bors
Auto merge of #38697 - alexcrichton:rollup, r=alexcrichton
7f2d2af 
Rollup of 25 pull requests

- Successful merges: #37149, #38491, #38517, #38559, #38587, #38609, #38611, #38622, #38628, #38630, #38631, #38632, #38635, #38647, #38649, #38655, #38659, #38660, #38662, #38665, #38671, #38674, #38676, #38693, #38695
- Failed merges: #38657, #38680
@bors bors merged commit ef8d5e1 into rust-lang:master Dec 30, 2016
@GuillaumeGomez GuillaumeGomez deleted the atomicptr_docs branch December 30, 2016 14:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@frewsxcv frewsxcv frewsxcv left review comments

Assignees

@frewsxcv frewsxcv

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GuillaumeGomez @frewsxcv @bors