RFR: 8283715: Update ObjectStreamClass to be final [v2]
Stuart Marks
smarks at openjdk.java.net
Tue Mar 29 16:14:41 UTC 2022
On Tue, 29 Mar 2022 16:07:18 GMT, Stuart Marks <smarks at openjdk.org> wrote:
>> Pretty much just what it says.
>
> Stuart Marks has updated the pull request incrementally with one additional commit since the last revision:
>
> Adjust nonfinal CSMs and rework error reporting in CheckCSMs.java test
Yes, the CheckCSMs test needed to be updated to remove ObjectStreamClass#forClass from the list of non-final CSMs, as that method is now part of a final class. The error message about ObjectStreamField#getType is misleading; I've fixed up the error reporting for this case.
It would indeed be interesting to make adjustments to ObjectStreamField. However, it's publicly subclassable, and it's not serializable, so a different set of dynamics apply. I'd like to consider that effort separately from this PR.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8009
More information about the core-libs-dev
mailing list