Merge pull request #22 from steffahn/improve_debug_implementation

Improve debug implementation

src/lib.rs

@@ -575,14 +575,10 @@ macro_rules! _impl_automatic_derive {
                 fmt: &mut core::fmt::Formatter,
             ) -> core::result::Result<(), core::fmt::Error> {
                 self.with_dependent(|owner, dependent| {
-                    write!(
-                        fmt,
-                        concat!(
-                            stringify!($StructName),
-                            " {{ owner: {:?}, dependent: {:?} }}"
-                        ),
-                        owner, dependent
-                    )
+                    fmt.debug_struct(stringify!($StructName))
+                        .field("owner", owner)
+                        .field("dependent", dependent)
+                        .finish()
                 })
             }
         }

tests/self_cell.rs

@@ -567,6 +567,29 @@ fn result_name_hygiene() {
     );
 }
 
+#[test]
+fn debug_impl() {
+    // See https://github.com/Voultapher/self_cell/pull/22
+    let ast_cell = PackedAstCell::new("xyz, abv".into(), |owner| owner.into());
+
+    assert_eq!(
+        format!("{:?}", &ast_cell),
+        "PackedAstCell { owner: \"xyz, abv\", dependent: Ast([\"z, \", \"yz\"]) }"
+    );
+
+    let hash_fmt = r#"PackedAstCell {
+    owner: "xyz, abv",
+    dependent: Ast(
+        [
+            "z, ",
+            "yz",
+        ],
+    ),
+}"#;
+
+    assert_eq!(format!("{:#?}", ast_cell), hash_fmt);
+}
+
 #[test]
 fn share_across_threads() {
     // drop_joined takes &mut self, so that's not a thread concern anyway.