Skip to content

Conversation

@projektir
Copy link
Contributor

@projektir projektir commented Apr 13, 2017

The same changes as PR #41240 , but for std::rc::Weak. At least, as far as I am aware, the Weak pointer is the same for both, and they're basically the same, just one is thread-safe and the other is not.

r? @alexcrichton

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Collaborator

bors commented Apr 13, 2017

📌 Commit f84cc0c has been approved by alexcrichton

@GuillaumeGomez
Copy link
Member

@bors: rollup

@bors
Copy link
Collaborator

bors commented Apr 13, 2017

⌛ Testing commit f84cc0c with merge c0088c9...

@bors
Copy link
Collaborator

bors commented Apr 13, 2017

💔 Test failed - status-appveyor

@TimNN
Copy link
Contributor

TimNN commented Apr 13, 2017

@bors retry

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 13, 2017
Updating docs for std::rc::Rc

The same changes as PR [rust-lang#41240 ](rust-lang#41240), but for [`std::rc::Weak`](https://doc.rust-lang.org/std/rc/struct.Weak.html). At least, as far as I am aware, the Weak pointer is the same for both, and they're basically the same, just one is thread-safe and the other is not.

r? @alexcrichton
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 13, 2017
Updating docs for std::rc::Rc

The same changes as PR [rust-lang#41240 ](rust-lang#41240), but for [`std::rc::Weak`](https://doc.rust-lang.org/std/rc/struct.Weak.html). At least, as far as I am aware, the Weak pointer is the same for both, and they're basically the same, just one is thread-safe and the other is not.

r? @alexcrichton
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 13, 2017
Updating docs for std::rc::Rc

The same changes as PR [rust-lang#41240 ](rust-lang#41240), but for [`std::rc::Weak`](https://doc.rust-lang.org/std/rc/struct.Weak.html). At least, as far as I am aware, the Weak pointer is the same for both, and they're basically the same, just one is thread-safe and the other is not.

r? @alexcrichton
bors added a commit that referenced this pull request Apr 13, 2017
Rollup of 3 pull requests

- Successful merges: #41240, #41250, #41266
- Failed merges:
@bors bors merged commit f84cc0c into rust-lang:master Apr 13, 2017
@projektir projektir deleted the weak_docs_rc branch April 13, 2017 21:00
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.

5 participants