Skip to content

syntax_pos::Symbol should not implement Sync #42466

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 7, 2017
Merged

syntax_pos::Symbol should not implement Sync #42466

merged 1 commit into from
Jun 7, 2017

Conversation

wesleywiser
Copy link
Member

Fixes #42407

@rust-highfive
Copy link
Contributor

r? @aturon

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

@arielb1
Copy link
Contributor

arielb1 commented Jun 6, 2017

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 6, 2017

📌 Commit c47df30 has been approved by arielb1

@arielb1 arielb1 added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jun 6, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Jun 7, 2017
@frewsxcv
Rollup merge of rust-lang#42466 - wesleywiser:fix_42407, r=arielb1
bedead2 
syntax_pos::Symbol should not implement Sync

Fixes rust-lang#42407
@frewsxcv frewsxcv mentioned this pull request Jun 7, 2017
Merged
bors added a commit that referenced this pull request Jun 7, 2017
@bors
Auto merge of #42494 - frewsxcv:rollup, r=frewsxcv
f73f675 
Rollup of 7 pull requests

- Successful merges: #42409, #42415, #42429, #42438, #42466, #42469, #42485
- Failed merges:
@bors bors merged commit c47df30 into rust-lang:master Jun 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aturon aturon

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@wesleywiser @rust-highfive @arielb1 @bors @aturon