Fix #2593: improve StackTraceElement deserialization

Author: cowtowncoder

release-notes/CREDITS

@@ -1,21 +1,34 @@
 Here are people who have contributed to the development of Jackson JSON processor
 databind core component, version 3.x
-Version numbers in brackets indicate release in which the problem was fixed
+(version numbers in brackets indicate release in which the problem was fixed)
 
 (note: for older credits, see `CREDITS-2.x` instead)
 
-Tatu Saloranta, [email protected]: author
+Author: Tatu Saloranta, [email protected]: author
 
-Alexander Koshman (akoshman@github)
+Co-Authors (with only partial listings below):
+
+* Joo Hyuk Kim (@JooHyukKim)
+* PJ Fanning (@pjfanning)
+* Sim Yih Tsern (@yihtsern)
+
+----------------------------------------------------------------------------
+
+Alexander Koshman (@akoshman)
  * Requested #1600: Serializing locale with underscore, not standard hyphen	
   [3.0.0]
 
-XakepSDK@github
+@XakepSDK
  * Requested #2013: Allow use of `java.nio.file.Path` for `readValue()`, `writeValue()`
   [3.0.0]
  * Requested #2411: `valueToTree()` during serialization (via `SerializerProvider()`)
   [3.0.0]
 
-Sven Döring (sdoeringNew@github)
+Sven Döring (@sdoeringNew)
  * Contributed #2013: Allow use of `java.nio.file.Path` for `readValue()`, `writeValue()`
   [3.0.0]
+
+Michael Dillon (@michaelcdillon)
+#2593: StackTraceElement w/ >= Java 9 doesn't deserialize properly
+ (requested by Michael D)
+

release-notes/VERSION

@@ -50,6 +50,8 @@ Versions: 3.x (for earlier see VERSION-2.x)
  (requested by XakepSDK@github)
 #2539: Add `Deserializers.hasDeserializerFor()` (and something for `DeserializerFactory`)
   to allow detection of explicitly supported types
+#2593: StackTraceElement w/ >= Java 9 doesn't deserialize properly
+ (requested by Michael D)
 #2713: Change wording of `UnrecognizedPropertyException` to refer to "property" not "field"
 #2828: Add `DatabindException` as intermediate subtype of `JsonMappingException`
 #3028: Change `UUIDSerializer` to use `StreamWriteCapability` check instead of

src/main/java/tools/jackson/databind/deser/jdk/StackTraceElementDeserializer.java

@@ -71,14 +71,17 @@ protected StackTraceElement constructValue(DeserializationContext ctxt,
     {
         // 21-May-2016, tatu: With Java 9, could use different constructor, probably
         //   via different module, and throw exception here if extra args passed
-        return new StackTraceElement(className, methodName, fileName, lineNumber);
+        // 08-Dec-2024, tatu: With Jackson 3.0 can use full Java 9 introduced
+        //   constructor, finally
+        return new StackTraceElement(classLoaderName, moduleName, moduleVersion,
+                className, methodName, fileName, lineNumber);
     }
 
     /**
      * Intermediate class used both for convenience of binding and
      * to support {@code PropertyNamingStrategy}.
      */
-    public final static class Adapter {
+    protected final static class Adapter {
         // NOTE: some String fields must not be nulls
         public String className = "", classLoaderName;
         public String declaringClass, format;