From 7736ed6c6251fcb3544d50c15290df6ac9542216 Mon Sep 17 00:00:00 2001 From: John Clements Date: Fri, 8 Feb 2013 18:58:33 -0800 Subject: [PATCH] json tidy --- src/libstd/json.rs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/libstd/json.rs b/src/libstd/json.rs index 07dbef64d2e..b6c4e3f11da 100644 --- a/src/libstd/json.rs +++ b/src/libstd/json.rs @@ -132,7 +132,7 @@ pub impl Encoder: serialize::Encoder { // other enums are encoded as vectors: // Kangaroo(34,"William") => ["Kangaroo",[34,"William"]] - + // the default expansion for enums is more verbose than I'd like; // specifically, the inner pair of brackets seems superfluous, // BUT the design of the enumeration framework and the requirements @@ -140,10 +140,11 @@ pub impl Encoder: serialize::Encoder { // be encoded "naked"--with no commas--and that the option name // can't be followed by just a comma, because there might not // be any elements in the tuple. - - // FIXME : this would be more precise and less frightening + + // this would be more precise and less frightening // with fully-qualified option names. To get that information, - // we'd have to change the expansion of auto-encode to pass those along. + // we'd have to change the expansion of auto-encode to pass + // those along. if (name == ~"Some") { f(); @@ -170,6 +171,7 @@ pub impl Encoder: serialize::Encoder { f(); self.wr.write_char(']'); } + fn emit_owned_vec(&self, len: uint, f: fn()) { self.emit_borrowed_vec(len, f) }