Skip to content

Add missing url in sync structs#40081

Merged
bors merged 1 commit into
rust-lang:masterfrom
GuillaumeGomez:poison-docs
Mar 2, 2017
Merged

Add missing url in sync structs#40081
bors merged 1 commit into
rust-lang:masterfrom
GuillaumeGomez:poison-docs

Conversation

@GuillaumeGomez

@GuillaumeGomez GuillaumeGomez commented Feb 24, 2017

Copy link
Copy Markdown
Member

@frewsxcv

frewsxcv commented Feb 25, 2017

Copy link
Copy Markdown
Contributor 
std/sync/struct.PoisonError.html:62: broken link fragment  `#method.lock` pointing to `std/sync/struct.RwLock.html`

std/sync/type.TryLockResult.html:54: broken link - std/sync/struct.LockResult.html

frewsxcv
frewsxcv reviewed Feb 28, 2017
View reviewed changes
Comment thread src/libstd/sys_common/poison.rs Outdated

@frewsxcv frewsxcv Feb 28, 2017

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.

Maybe you meant https://doc.rust-lang.org/std/sync/struct.Condvar.html#method.wait

@GuillaumeGomez GuillaumeGomez Feb 28, 2017

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Absolutely!

@GuillaumeGomez

GuillaumeGomez commented Mar 1, 2017

Copy link
Copy Markdown
Member Author

Updated.

@frewsxcv

frewsxcv commented Mar 2, 2017

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Mar 2, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 992dd20 has been approved by frewsxcv

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Mar 2, 2017
@GuillaumeGomez
Rollup merge of rust-lang#40081 - GuillaumeGomez:poison-docs, r=frewsxcv
5d08775 
Add missing url in sync structs

r? @frewsxcv
@GuillaumeGomez GuillaumeGomez mentioned this pull request Mar 2, 2017
Merged
bors added a commit that referenced this pull request Mar 2, 2017
@bors
Auto merge of #40206 - GuillaumeGomez:rollup, r=GuillaumeGomez
8ae411e 
Rollup of 9 pull requests

- Successful merges: #40081, #40144, #40168, #40169, #40170, #40173, #40175, #40191, #40194
- Failed merges:
@bors bors merged commit 992dd20 into rust-lang:master Mar 2, 2017
@GuillaumeGomez GuillaumeGomez deleted the poison-docs branch March 2, 2017 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@frewsxcv frewsxcv frewsxcv left review comments

Reviewers whose approvals may not affect merge requirements

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