Skip to content

Allow #[derive()] to generate unsafe methods #25524

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
May 17, 2015
Merged

Allow #[derive()] to generate unsafe methods #25524

merged 1 commit into from
May 17, 2015

Conversation

Manishearth
Copy link
Member

No description provided.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@Manishearth
Copy link
Member Author

r? @nrc
or @huonw

@rust-highfive rust-highfive assigned nrc and unassigned nikomatsakis May 17, 2015
@emberian
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented May 17, 2015

📌 Commit 86915e5 has been approved by cmr

@Manishearth Manishearth mentioned this pull request May 17, 2015
Closed
@bors
Copy link
Collaborator

bors commented May 17, 2015

📌 Commit e00e7c9 has been approved by cmr

@Manishearth
Copy link
Member Author

@bors: r=cmr e00e7c9 rollup

@bors
Copy link
Collaborator

bors commented May 17, 2015

⌛ Testing commit e00e7c9 with merge 86aced4...

@bors
Copy link
Collaborator

bors commented May 17, 2015

💔 Test failed - auto-linux-64-x-android-t

@Manishearth
Copy link
Member Author

@bors: r=cmr

Argh, git add

@bors
Copy link
Collaborator

bors commented May 17, 2015

📌 Commit fe5f84e has been approved by cmr

@bors
Copy link
Collaborator

bors commented May 17, 2015

⌛ Testing commit fe5f84e with merge de7080f...

@bors
Copy link
Collaborator

bors commented May 17, 2015

💔 Test failed - auto-linux-64-x-android-t

@Manishearth
Copy link
Member Author

@bors: r=cmr

@bors
Copy link
Collaborator

bors commented May 17, 2015

📌 Commit 5e10576 has been approved by cmr

@bors
Copy link
Collaborator

bors commented May 17, 2015

⌛ Testing commit 5e10576 with merge 6bdee64...

@bors
Copy link
Collaborator

bors commented May 17, 2015

💔 Test failed - auto-mac-64-nopt-t

@Manishearth
Copy link
Member Author

@bors: r=cmr

@bors
Copy link
Collaborator

bors commented May 17, 2015

📌 Commit 5b63841 has been approved by cmr

bors added a commit that referenced this pull request May 17, 2015
@bors
Auto merge of #25524 - Manishearth:unsafe_derive, r=cmr
8b7c17d
@bors
Copy link
Collaborator

bors commented May 17, 2015

⌛ Testing commit 5b63841 with merge 8b7c17d...

@bors
Copy link
Collaborator

bors commented May 17, 2015

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-32-nopt-t, auto-win-32-opt, auto-win-64-nopt-t, auto-win-64-opt

@bors bors merged commit 5b63841 into rust-lang:master May 17, 2015
@Manishearth Manishearth deleted the unsafe_derive branch May 17, 2015 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nrc nrc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Manishearth @rust-highfive @emberian @bors @nikomatsakis @nrc