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

Add CFPropertyList serialization #88

Closed
wants to merge 2 commits into from
Closed

Conversation

@ghost
Copy link

ghost commented Jan 2, 2017

Includes passing tests. rustfmt touched a lot of the code, I can fix this if it's an issue.


This change is Reviewable

@ghost ghost changed the title Add CFPropertyList Add CFPropertyList serializat Jan 2, 2017
@ghost ghost changed the title Add CFPropertyList serializat Add CFPropertyList serialization Jan 2, 2017
@emilio
Copy link
Member

emilio commented Jan 2, 2017

Can you split the formatting changes and the rest of changes in different commits? That'll make it easier to review :)

@bors-servo r? @jdm

@ghost
Copy link
Author

ghost commented Jan 2, 2017

Done! 😄

Christian Howe added 2 commits Jan 2, 2017
@ghost
Copy link
Author

ghost commented Jan 2, 2017

Reopening on a feature branch

@ghost ghost closed this Jan 2, 2017
@ghost ghost mentioned this pull request Jan 2, 2017
bors-servo added a commit that referenced this pull request Jan 5, 2017
Add CFPropertyList serialization

Tests included. Reopened from #88

<!-- 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/90)
<!-- Reviewable:end -->
jdm pushed a commit that referenced this pull request Feb 1, 2018
Add CGWindowListCreateImageFromArray()

Just adds a binding to one small function.

<!-- 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-graphics-rs/88)
<!-- Reviewable:end -->
This issue was closed.
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

1 participant
You can’t perform that action at this time.