Skip to content

derive: use intrinsics::unreachable over unreachable!() - #32250

Merged
alexcrichton merged 2 commits into
rust-lang:masterfrom
durka:derive-31574
Mar 15, 2016
Merged

derive: use intrinsics::unreachable over unreachable!()#32250
alexcrichton merged 2 commits into
rust-lang:masterfrom
durka:derive-31574

Conversation

@durka

@durka durka commented Mar 14, 2016

Copy link
Copy Markdown
Contributor

derive: use intrinsics::unreachable over unreachable!()

Fixes #31574.

Spawned from #32139.

r? @alexcrichton

@durka

durka commented Mar 14, 2016

Copy link
Copy Markdown
Contributor Author

The commit from #32139 that factors out the construction of intrinsics::unreachable() depends on both this PR and #32252 so it will have to wait.

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ a09a419

@bors

bors commented Mar 15, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit a09a419 with merge f9121e8...

bors added a commit that referenced this pull request Mar 15, 2016
derive: use intrinsics::unreachable over unreachable!()

derive: use intrinsics::unreachable over unreachable!()

Fixes #31574.

Spawned from #32139.

r? @alexcrichton
@alexcrichton

Copy link
Copy Markdown
Member

This passed in bors but I'm messing with the configuration so it didn't realize it, so merging manually.

@alexcrichton
alexcrichton merged commit a09a419 into rust-lang:master Mar 15, 2016
@strega-nil

Copy link
Copy Markdown
Contributor

yay!

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