Streamline ThrowableDeserializer slightly to reduce call stack for deep nesting (#4345)

cowtowncoder · web-flow · commit 62f869b7a1b3 · 2024-01-26T09:27:52.000-08:00
diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/std/ThrowableDeserializer.java b/src/main/java/com/fasterxml/jackson/databind/deser/std/ThrowableDeserializer.java
@@ -157,9 +157,10 @@ public Object deserializeFromObject(JsonParser p, DeserializationContext ctxt) t
                 if (p.hasToken(JsonToken.VALUE_NULL)) {
                     suppressed = null;
                 } else {
-                    // Resolve Throwable[] to JavaType pro-actively
-                    suppressed = ctxt.readValue(p,
+                    // Inlined `DeserializationContext.readValue()` to minimize call depth
+                    JsonDeserializer<Object> deser = ctxt.findRootValueDeserializer(
                             ctxt.constructType(Throwable[].class));
+                    suppressed = (Throwable[]) deser.deserialize(p, ctxt);
                 }
                 continue;
             }

Original file line number	Diff line number	Diff line change
`@@ -157,9 +157,10 @@ public Object deserializeFromObject(JsonParser p, DeserializationContext ctxt) t`
`157`	`157`	`if (p.hasToken(JsonToken.VALUE_NULL)) {`
`158`	`158`	`suppressed = null;`
`159`	`159`	`} else {`
`160`		`- // Resolve Throwable[] to JavaType pro-actively`
`161`		`- suppressed = ctxt.readValue(p,`
	`160`	+ // Inlined `DeserializationContext.readValue()` to minimize call depth
	`161`	`+ JsonDeserializer<Object> deser = ctxt.findRootValueDeserializer(`
`162`	`162`	`ctxt.constructType(Throwable[].class));`
	`163`	`+ suppressed = (Throwable[]) deser.deserialize(p, ctxt);`
`163`	`164`	`}`
`164`	`165`	`continue;`
`165`	`166`	`}`