Skip to content

don't unwrap non-local span #36713

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
Sep 25, 2016
Merged

don't unwrap non-local span #36713

merged 1 commit into from
Sep 25, 2016

Conversation

TimNN
Copy link
Contributor

@TimNN TimNN commented Sep 25, 2016

Fixes #36708.

@rust-highfive
Copy link
Contributor

r? @Aatch

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

@arielb1
Copy link
Contributor

arielb1 commented Sep 25, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Sep 25, 2016

📌 Commit 47b918d has been approved by arielb1

@bors
Copy link
Collaborator

bors commented Sep 25, 2016

⌛ Testing commit 47b918d with merge 2d1d4d1...

bors added a commit that referenced this pull request Sep 25, 2016
@bors
Auto merge of #36713 - TimNN:fix-36708, r=arielb1
2d1d4d1 
don't unwrap non-local span

Fixes #36708.
@bors
Copy link
Collaborator

bors commented Sep 25, 2016

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-cargotest, auto-linux-64-cross-freebsd, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-opt-rustbuild, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-opt, auto-mac-64-opt-rustbuild, auto-mac-cross-ios-opt, auto-win-gnu-32-opt, auto-win-gnu-32-opt-rustbuild, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-cargotest, auto-win-msvc-64-opt, auto-win-msvc-64-opt-rustbuild
Approved by: arielb1
Pushing 2d1d4d1 to master...

@bors bors merged commit 47b918d into rust-lang:master Sep 25, 2016
@TimNN TimNN deleted the fix-36708 branch September 25, 2016 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Aatch Aatch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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