Skip to content
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

Use $crate in core::panic! #27385

Merged
merged 1 commit into from
Jul 30, 2015
Merged

Conversation

chris-morgan
Copy link
Member

I’ve been sitting on this one for ages now. Silly me, if only I had got on and submitted it earlier it’d be into the stable release by now…

@rust-highfive
Copy link
Collaborator

r? @brson

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

@brson
Copy link
Contributor

brson commented Jul 29, 2015

@alexcrichton is there any backcompat hazard here?

@alexcrichton
Copy link
Member

Yeah this is fine because the macro isn't reexported in std anyway (std has its own panic macro)

@bors: r+ fc5b79c

@bors
Copy link
Contributor

bors commented Jul 30, 2015

⌛ Testing commit fc5b79c with merge 5fcaf95...

bors added a commit that referenced this pull request Jul 30, 2015
…hton

I’ve been sitting on this one for ages now. Silly me, if only I had got on and submitted it earlier it’d be into the stable release by now…
@bors bors merged commit fc5b79c into rust-lang:master Jul 30, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants