Skip to content

Commit 351d847

Browse files
committed
Omit a layer of unnecessary nesting from Debug impl
1 parent baae6d9 commit 351d847

File tree

2 files changed

+19
-15
lines changed

2 files changed

+19
-15
lines changed

src/value/mod.rs

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,16 @@ impl Debug for Value {
177177
Value::Bool(v) => formatter.debug_tuple("Bool").field(&v).finish(),
178178
Value::Number(ref v) => Debug::fmt(v, formatter),
179179
Value::String(ref v) => formatter.debug_tuple("String").field(v).finish(),
180-
Value::Array(ref v) => formatter.debug_tuple("Array").field(v).finish(),
181-
Value::Object(ref v) => formatter.debug_tuple("Object").field(v).finish(),
180+
Value::Array(ref v) => {
181+
formatter.write_str("Array(")?;
182+
Debug::fmt(v, formatter)?;
183+
formatter.write_str(")")
184+
}
185+
Value::Object(ref v) => {
186+
formatter.write_str("Object(")?;
187+
Debug::fmt(v, formatter)?;
188+
formatter.write_str(")")
189+
}
182190
}
183191
}
184192
}

tests/debug.rs

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -50,20 +50,16 @@ fn error() {
5050
assert_eq!(format!("{:?}", err), expected);
5151
}
5252

53-
const INDENTED_EXPECTED: &str = r#"Object(
54-
{
55-
"array": Array(
56-
[
57-
Number(
58-
0,
59-
),
60-
Number(
61-
1,
62-
),
63-
],
53+
const INDENTED_EXPECTED: &str = r#"Object({
54+
"array": Array([
55+
Number(
56+
0,
6457
),
65-
},
66-
)"#;
58+
Number(
59+
1,
60+
),
61+
]),
62+
})"#;
6763

6864
#[test]
6965
fn indented() {

0 commit comments

Comments
 (0)