On Tue, 26 Nov 2024 13:04:41 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:
> Please review this PR which adds the `final` modifier to non-subclassable > classes in `java.base`. > > The classes were identified using an automated analysis. See CSR for details. > > Besides simply adding the `final` access modifier, the PR: > > * Updates a note in `java.lang.constant.DynamicCallSiteDesc` to not reference > subtypes. See CSR for discussion. > * Removes the class `java.lang.Runtime` from the test > `test/jdk/jdk/internal/reflect/CallerSensitive/CheckCSMs.java` > * Updates the copyright year of affected source files src/java.base/share/classes/java/lang/module/ModuleDescriptor.java line 1318: > 1316: Version version, > 1317: Set<Modifier> modifiers, > 1318: Set<Requires> requires, I assume you'll fix the alignment if this goes ahead. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22389#discussion_r1859076834
