Simplify serialize::{En,De}coder, add support for serializing maps, and other misc fns #5630

Closed
wants to merge 22 commits into
from

Projects

None yet

4 participants

@erickt

@nikomatsakis and I were talking about how the serializers were a bit too complicated. None of the users of With the emit_option and read_option functions, the serializers are now moving more high level. This patch series continues that trend. I've removed support for emitting specific string and vec types, and added support for emitting mapping types.

@nikomatsakis

I'd give this r+, but it's already bit-rotted...

@erickt

@nikomatsakis: It's back to mergable. I also fleshed out serialization support for the other container types.

@pcwalton

r+

carrying forward from @nikomatsakis

saw approval from pcwalton
at erickt@909d8f0

merging erickt/rust/serial = 909d8f0 into auto

erickt/rust/serial = 909d8f0 merged ok, testing candidate = 44de87cf

@erickt

@bors: retry

@erickt

r+

saw approval from erickt
at erickt@810c4d8

merging erickt/rust/serial = 810c4d8 into auto

erickt/rust/serial = 810c4d8 merged ok, testing candidate = 6dd20c8

fast-forwarding incoming to auto = 6dd20c8

@bors bors added a commit that referenced this pull request Mar 30, 2013
@bors bors auto merge of #5630 : erickt/rust/serial, r=erickt
@nikomatsakis and I were talking about how the serializers were a bit too complicated. None of the users of With the `emit_option` and `read_option` functions, the serializers are now moving more high level. This patch series continues that trend. I've removed support for emitting specific string and vec types, and added support for emitting mapping types.
6dd20c8
@bors bors closed this Mar 30, 2013
@jayanderson jayanderson pushed a commit that referenced this pull request Nov 11, 2013
@bors bors auto merge of #5630 : erickt/rust/serial, r=erickt
@nikomatsakis and I were talking about how the serializers were a bit too complicated. None of the users of With the `emit_option` and `read_option` functions, the serializers are now moving more high level. This patch series continues that trend. I've removed support for emitting specific string and vec types, and added support for emitting mapping types.
e474750
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment