RFR: 8344943: Mark not subclassable classes final in java.base exported classes

Chen Liang liach at openjdk.org
Thu Jan 16 20:42:53 UTC 2025


On Tue, 26 Nov 2024 19:08:26 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

>> Thanks! Yes, that's good feedback. There are some non-API changes that I'd like to make, such as this and updating the comment for the `InterfaceAddress` constructor.
>> 
>> I prefered to make the initial PR draft contain just the CSR changes, such that PR would be useful and clean as a Specification change, since the diff is a bit noisy.
>> 
>> I'll keep your review comment here unresolved for now and get back to it pending CSR approval.
>
> For the record:
> 
> It was not completely clear to me whether restricting these two package-private constructors to private is considered a specification change in the CSR sense. I opted to point them out in the CSR, just to make the change abundantly clear.

This constructor is not part of the public API: package-private is not accessible, and it does not appear in the API docs: https://docs.oracle.com/en/java/javase/23/docs/api/java.base/java/lang/module/ModuleDescriptor.html

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22389#discussion_r1859114697


More information about the net-dev mailing list