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

Reduce usage of libc crate. #156

Merged
merged 1 commit into from Feb 2, 2018
Merged

Conversation

@waywardmonkeys
Copy link
Contributor

waywardmonkeys commented Feb 1, 2018

Switch from using libc types to ::std::os::raw. This is a breaking
change due to the differences in how c_void is defined.

core-foundation-sys no longer needs the libc dependency.


This change is Reviewable

@jdm
jdm approved these changes Feb 1, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Feb 1, 2018

The latest upstream changes (presumably #157) made this pull request unmergeable. Please resolve the merge conflicts.

Switch from using `libc` types to `::std::os::raw`. This is a breaking
change due to the differences in how `c_void` is defined.

`core-foundation-sys` no longer needs the `libc` dependency.
@waywardmonkeys waywardmonkeys force-pushed the waywardmonkeys:reduce-libc-usage branch from da7c35b to 7e7d65b Feb 1, 2018
@waywardmonkeys
Copy link
Contributor Author

waywardmonkeys commented Feb 1, 2018

Updated (and force pushed) with merge conflicts addressed.

jdm pushed a commit that referenced this pull request Feb 1, 2018
Update core-graphics to 0.7

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

jdm commented Feb 2, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2018

📌 Commit 7e7d65b has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2018

Testing commit 7e7d65b with merge fb8a43f...

bors-servo added a commit that referenced this pull request Feb 2, 2018
Reduce usage of libc crate.

Switch from using `libc` types to `::std::os::raw`. This is a breaking
change due to the differences in how `c_void` is defined.

`core-foundation-sys` no longer needs the `libc` dependency.

<!-- 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/156)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2018

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

@bors-servo bors-servo merged commit 7e7d65b into servo:master Feb 2, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@waywardmonkeys waywardmonkeys deleted the waywardmonkeys:reduce-libc-usage branch Feb 4, 2018
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.