Skip to content

Tweak "unecessary unsafe block" error spans #46805

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 19, 2017
Merged

Tweak "unecessary unsafe block" error spans #46805

merged 1 commit into from
Dec 19, 2017

Conversation

estebank
Copy link
Contributor

No description provided.

@rust-highfive
Copy link
Contributor

r? @michaelwoerister

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

@michaelwoerister
Copy link
Member

r? @arielb1

I think @arielb1 did some work there recently. Re-assigning.

@estebank estebank force-pushed the unnecessary-unsafe-spans branch from 360fb98 to 7781a13 Compare December 18, 2017 16:51
@estebank estebank force-pushed the unnecessary-unsafe-spans branch from 7781a13 to e1dc7ac Compare December 18, 2017 19:11
@arielb1
Copy link
Contributor

arielb1 commented Dec 18, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 18, 2017

📌 Commit e1dc7ac has been approved by arielb1

@arielb1
Copy link
Contributor

arielb1 commented Dec 18, 2017

@bors rollup

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Dec 18, 2017
@GuillaumeGomez
Rollup merge of rust-lang#46805 - estebank:unnecessary-unsafe-spans, …
325d9fb 
…r=arielb1

Tweak "unecessary unsafe block" error spans
@GuillaumeGomez GuillaumeGomez mentioned this pull request Dec 18, 2017
Merged
bors added a commit that referenced this pull request Dec 19, 2017
@bors
Auto merge of #46829 - GuillaumeGomez:rollup, r=GuillaumeGomez
c8b94c6 
Rollup of 11 pull requests

- Successful merges: #46700, #46786, #46790, #46800, #46801, #46802, #46804, #46805, #46812, #46824, #46825
- Failed merges:
@bors bors merged commit e1dc7ac into rust-lang:master Dec 19, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@arielb1 arielb1

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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