On Thu, 21 Dec 2023 21:24:43 GMT, Roger Riggs <[email protected]> wrote:
>> Raffaello Giulietti has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Corrected @Label of event and of field.
>
> src/java.base/share/classes/java/io/ObjectStreamClass.java line 466:
>
>> 464:
>> 465: if (SerializationMisdeclarationEvent.enabled() && serializable)
>> {
>> 466: new
>> SerializationMisdeclarationChecker(cl).checkMisdeclarations();
>
> Is there any benefit to avoiding the allocation and passing the class through
> the methods as an extra arg?
The allocation has been removed and the event messages have been simplified in
the most recent commit.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17129#discussion_r1434558624
