about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2014-06-02 01:06:39 -0700
committerbors <bors@rust-lang.org>2014-06-02 01:06:39 -0700
commit46dad765f047ae5cd49a4b6e509ab726c48838c8 (patch)
tree9e5141d69692bb35779be040497206c6938c8ce8
parentb981add9ee56a2d6dc11aa48f01aac5d0dda9327 (diff)
parent1dc13e4ad4a140ae1ac4d84e13411c7406196926 (diff)
downloadrust-46dad765f047ae5cd49a4b6e509ab726c48838c8.tar.gz
rust-46dad765f047ae5cd49a4b6e509ab726c48838c8.zip
auto merge of #14596 : Sawyer47/rust/encodable-fix, r=alexcrichton
Closes #14021
-rw-r--r--src/libsyntax/ext/deriving/encodable.rs8
-rw-r--r--src/test/run-pass/issue-14021.rs29
2 files changed, 37 insertions, 0 deletions
diff --git a/src/libsyntax/ext/deriving/encodable.rs b/src/libsyntax/ext/deriving/encodable.rs
index d6b7e84b535..4f92dff0c59 100644
--- a/src/libsyntax/ext/deriving/encodable.rs
+++ b/src/libsyntax/ext/deriving/encodable.rs
@@ -175,6 +175,14 @@ fn encodable_substructure(cx: &mut ExtCtxt, trait_span: Span,
                 stmts.push(cx.stmt_expr(call));
             }
 
+            // unit structs have no fields and need to return Ok()
+            if stmts.is_empty() {
+                let ret_ok = cx.expr(trait_span,
+                                     ExprRet(Some(cx.expr_ok(trait_span,
+                                                             cx.expr_lit(trait_span, LitNil)))));
+                stmts.push(cx.stmt_expr(ret_ok));
+            }
+
             let blk = cx.lambda_stmts_1(trait_span, stmts, blkarg);
             cx.expr_method_call(trait_span,
                                 encoder,
diff --git a/src/test/run-pass/issue-14021.rs b/src/test/run-pass/issue-14021.rs
new file mode 100644
index 00000000000..f25a1fcdab6
--- /dev/null
+++ b/src/test/run-pass/issue-14021.rs
@@ -0,0 +1,29 @@
+// Copyright 2014 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+
+extern crate serialize;
+
+use serialize::{Encodable, Decodable};
+use serialize::json;
+
+#[deriving(Encodable, Decodable, PartialEq, Show)]
+struct UnitLikeStruct;
+
+pub fn main() {
+    let obj = UnitLikeStruct;
+    let json_str: String = json::Encoder::str_encode(&obj);
+
+    let json_object = json::from_str(json_str.as_slice());
+    let mut decoder = json::Decoder::new(json_object.unwrap());
+    let mut decoded_obj: UnitLikeStruct = Decodable::decode(&mut decoder).unwrap();
+
+    assert_eq!(obj, decoded_obj);
+}