RFR: 8328874: Class::forName0 should validate the class name length early [v8]

Guanqiang Han ghan at openjdk.org
Thu Aug 28 00:48:50 UTC 2025


On Wed, 27 Aug 2025 17:52:11 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Guanqiang Han has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   change copyright year
>
> src/java.base/share/classes/java/lang/Class.java line 95:
> 
>> 93: import sun.reflect.annotation.*;
>> 94: 
>> 95: import static jdk.internal.util.ModifiedUtf.utfLen;
> 
> Do not use static imports across package boundaries, it make the code harder to read and to know where that method came from.

Hi @RogerRiggs , thank you for pointing out this detail. I’ve updated the PR accordingly and will pay more attention to such readability issues in future changes. Please have another look.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26802#discussion_r2305755787


More information about the core-libs-dev mailing list