On Thu, 4 Jun 2026 11:31:41 GMT, Viktor Klang <[email protected]> wrote:

>> src/java.base/share/classes/java/io/ObjectStreamClass.java line 197:
>> 
>>> 195:     private MethodHandle deserializationCtr;
>>> 196:     /** value deserialization method identified by {@link 
>>> DeserializeConstructor}.*/
>>> 197:     private Executable deserializeConstructor;
>> 
>> These two fields seems confusingly similarly named, possible to 
>> differentiate the names a bit for clarity?
>
> Also, can a type have both a `deserializationCtr` AND a 
> `deserializeConstructor` or are they mutually exclusive?

I named this `deserializeConstructor` because this is for 
`@DeserializeConstructor`. Maybe we can rename both the annotation and the 
field "deserialization factory" or some other better name instead?

-------------

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2326#discussion_r3355869594

Reply via email to