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

Fix RwLock*Guard::map to not allow escaping a reference to the data. #31440

Merged
merged 1 commit into from
Feb 7, 2016

Conversation

reem
Copy link
Contributor

@reem reem commented Feb 6, 2016

Also update the instability reason to include a note about a possible
bad interaction with condition variables on systems that allow
waiting on a RwLock guard.

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

Looks like tidy doesn't like the line lengths here, but otherwise lgtm

Also update the instability reason to include a note about a possible
bad interaction with condition variables on systems that allow
waiting on a RwLock guard.
@reem
Copy link
Contributor Author

reem commented Feb 6, 2016

Updated.

@alexcrichton
Copy link
Member

@bors: r+ ad73330

@bors
Copy link
Contributor

bors commented Feb 7, 2016

⌛ Testing commit ad73330 with merge 3623797...

bors added a commit that referenced this pull request Feb 7, 2016
Also update the instability reason to include a note about a possible
bad interaction with condition variables on systems that allow
waiting on a RwLock guard.
@bors bors merged commit ad73330 into rust-lang:master Feb 7, 2016
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.

4 participants