don’t use debug output is Display impl for CborValue

fixes #14

src/canonical.rs

@@ -145,14 +145,12 @@ fn canonicalise_dict<'a>(bytes: &'a [u8], builder: &mut DictWriter) -> Option<&'
                 builder.with_key(key, |b| b.write_neg(neg, tags));
             }
             MAJOR_BYTES => {
+                let decoded = update(bytes, value_bytes(b, false))?;
                 if tags.single() == Some(TAG_CBOR_ITEM) {
-                    // drop CBOR item encoding wrapper - may choose to use these later for more efficient skipping
-                    let decoded = update(bytes, value_bytes(b, false))?;
                     // the line above has advanced the main loop’s reference, here we advance a temporary one
                     one(&mut decoded.as_ref(), key, builder)?
                 } else {
-                    let value = update(bytes, value_bytes(b, false))?;
-                    builder.with_key(key, |b| b.write_bytes(value.as_ref(), tags));
+                    builder.with_key(key, |b| b.write_bytes(decoded.as_ref(), tags));
                 }
             }
             MAJOR_STR => {

src/lib.rs

@@ -107,7 +107,7 @@ impl<'a> Cbor<'a> {
 
 impl<'a> AsRef<[u8]> for Cbor<'a> {
     fn as_ref(&self) -> &[u8] {
-        &self.0
+        self.0
     }
 }
 

src/value.rs

@@ -61,7 +61,7 @@ impl<'a> Display for ValueKind<'a> {
                 if *x == 0f64 && x.is_sign_negative() {
                     write!(f, "-0.0")
                 } else {
-                    write!(f, "{:?}", x)
+                    write!(f, "{}", x)
                 }
             }
             Str(s) => write!(f, "\"{}\"", s.escape_debug()),
@@ -408,10 +408,21 @@ fn bytes_to_float(bytes: &[u8]) -> f64 {
 
 #[cfg(test)]
 mod tests {
-    use crate::CborOwned;
+    use crate::{CborBuilder, CborOwned, Encoder};
 
     use super::*;
 
+    #[test]
+    fn display() {
+        fn to_cbor_str(f: f64) -> String {
+            format!("{}", CborBuilder::new().encode_f64(f))
+        }
+        assert_eq!(to_cbor_str(1.0), "1");
+        assert_eq!(to_cbor_str(-1.1), "-1.1");
+        assert_eq!(to_cbor_str(0.0), "0");
+        assert_eq!(to_cbor_str(-0.0), "-0.0");
+    }
+
     #[test]
     fn base64string() {
         fn to_cbor(s: &str, tag: u64) -> CborOwned {