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

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

Consistently use std::os::raw::c_void instead of libc::c_void #214

Merged
merged 1 commit into from May 7, 2018

Conversation

@jrmuizel
Copy link
Collaborator

jrmuizel commented May 6, 2018

The major versions of cocoa and core-graphics are bumped as I believe
they are the only crates that meaningfully expose c_void.

Fixes #212


This change is Reviewable

The major versions of cocoa and core-graphics are bumped as I believe
they are the only crates that meaningfully expose c_void.

Fixes #212
@jdm
Copy link
Member

jdm commented May 7, 2018

@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2018

📌 Commit 6457799 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2018

Testing commit 6457799 with merge ec288cd...

bors-servo added a commit that referenced this pull request May 7, 2018
Consistently use std::os::raw::c_void instead of libc::c_void

The major versions of cocoa and core-graphics are bumped as I believe
they are the only crates that meaningfully expose c_void.

Fixes #212

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/214)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2018

☀️ Test successful - status-travis
Approved by: jdm
Pushing ec288cd to master...

@bors-servo bors-servo merged commit 6457799 into servo:master May 7, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.