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

Allow serialized types to use a more compact representation ... #217

Merged
merged 1 commit into from Nov 20, 2017

Conversation

@Marwes
Copy link
Contributor

Marwes commented Nov 2, 2017

... by utilizing that bincode is not human readable.

Uses the changes in serde-rs/serde#1044 which
allows data formats to report that they are not human readable. This
lets certain types serialize themselves into a more compact form as they
know that the serialized form does not need to be readable.

Closes #215

BREAKING CHANGE

This changes how types serialize themselves if they detect the
is_human_readable state.

... by utilizing that bincode is not human readable.

Uses the changes in serde-rs/serde#1044 which
allows data formats to report that they are not human readable. This
lets certain types serialize themselves into a more compact form as they
know that the serialized form does not need to be readable.

Closes #215

BREAKING CHANGE

This changes how types serialize themselves if they detect the
`is_human_readable` state.
@TyOverby
Copy link
Collaborator

TyOverby commented Nov 2, 2017

I'm going to hold off on merging this until I talk to some people that are using bincode. I only know of one group that is using bincode and storing the data on disk.

@Marwes
Copy link
Contributor Author

Marwes commented Nov 20, 2017

@TyOverby Any movement on this?

@TyOverby
Copy link
Collaborator

TyOverby commented Nov 20, 2017

No reason for this to not be merged right now, but I'm going to hold off on publishing until I land the other breaking changes.

This change will likely be a part of 1.0.

@TyOverby TyOverby merged commit 0867029 into servo:master Nov 20, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Marwes Marwes deleted the Marwes:non_human_readable branch Nov 20, 2017
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

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