Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

Export serde internal Serializer and Deserializer #129

Closed
wants to merge 2 commits into from

Conversation

pickfire
Copy link

@pickfire pickfire commented Apr 28, 2019

Fix #44.

  • Add benchmarks
  • Split internal Serializer
  • Add public Serializer with Writer
  • Split internal Deserializer
  • Add public Deserializer with Reader

Blocking on chyh1990/yaml-rust#127 and probably chyh1990/yaml-rust#33 as well.

@pickfire pickfire changed the title WIP: Split serde Serializer internal Export serde internal Serializer and Deserializer Apr 28, 2019
type SerializeStructVariant = &'a mut Serializer<W>;

#[inline]
fn serialize_bool(self, value: bool) -> Result<()> {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dtolnay I wonder if this is the correct method to do this or we should patch it to chyh1990/yaml-rust#127?

Copy link
Owner

@dtolnay dtolnay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closing because I don't think this is going to be the right approach. Per #44 (comment), I would prefer to switch away from yaml-rust as a yaml backend before attempting this change.

Thanks anyway!

@dtolnay dtolnay closed this Oct 4, 2019
@pickfire pickfire deleted the feature-expose branch October 5, 2019 14:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose the Serializer and Deserializer types
2 participants